Skip to content

feat(v9): migrate keyboard-keys,priority-overflow,alert,aria,avatar,button,conf-griffel,context-selector to ship rolluped only dts#22823

Merged
Hotell merged 2 commits intomicrosoft:masterfrom
Hotell:hotell/build/migrate-v9-to-rolluped-dts-p4
May 5, 2022
Merged

feat(v9): migrate keyboard-keys,priority-overflow,alert,aria,avatar,button,conf-griffel,context-selector to ship rolluped only dts#22823
Hotell merged 2 commits intomicrosoft:masterfrom
Hotell:hotell/build/migrate-v9-to-rolluped-dts-p4

Conversation

@Hotell
Copy link
Contributor

@Hotell Hotell commented May 4, 2022

Applied yarn workspace-generator migrate-converged-pkg to ship only rolluped type definitions for:

  • keyboard-keys,
  • priority-overflow,
  • alert,
  • aria,
  • avatar,
  • button,
  • conformance-griffel,
  • context-selector

Related Issue(s)

Fixes partially #22429

@Hotell Hotell changed the title Hotell/build/migrate v9 to rolluped dts p4 feat(v9): migrate keyboard-keys,priority-overflow,alert,aria,avatar,button,conf-griffel,context-selector to ship rolluped only dts May 4, 2022
@Hotell Hotell mentioned this pull request May 4, 2022
50 tasks
@codesandbox-ci
Copy link

codesandbox-ci bot commented May 4, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 9d3e021:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@fabricteam
Copy link
Collaborator

fabricteam commented May 4, 2022

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-accordion
Accordion (including children components)
73.754 kB
22.494 kB
react-badge
Badge
20.965 kB
6.605 kB
react-badge
CounterBadge
21.918 kB
6.919 kB
react-badge
PresenceBadge
22.005 kB
6.582 kB
react-combobox
Combobox
60.564 kB
20.751 kB
react-divider
Divider
15.455 kB
5.57 kB
react-radio
Radio
29.382 kB
10.047 kB
react-radio
RadioGroup
13.61 kB
5.462 kB
react-switch
Switch
25.312 kB
8.202 kB
react-textarea
Textarea
20.672 kB
7.07 kB
🤖 This report was generated against 8437eb3b6023bdac5b234538e1116b1c7d24f53f

@size-auditor
Copy link

size-auditor bot commented May 4, 2022

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 8437eb3b6023bdac5b234538e1116b1c7d24f53f (build)

@Hotell
Copy link
Contributor Author

Hotell commented May 4, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@Hotell Hotell marked this pull request as ready for review May 4, 2022 15:30
@Hotell Hotell requested review from a team, behowell and khmakoto as code owners May 4, 2022 15:30
@Hotell
Copy link
Contributor Author

Hotell commented May 4, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Hotell added 2 commits May 4, 2022 17:34
@Hotell Hotell force-pushed the hotell/build/migrate-v9-to-rolluped-dts-p4 branch from 52f8acd to 9d3e021 Compare May 4, 2022 15:34
@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1116 1119 5000
Button mount 673 713 5000
FluentProvider mount 2287 2292 5000
FluentProviderWithTheme mount 341 354 10
FluentProviderWithTheme virtual-rerender 299 289 10
FluentProviderWithTheme virtual-rerender-with-unmount 385 382 10
MakeStyles mount 1924 1930 50000

@Hotell Hotell merged commit e465622 into microsoft:master May 5, 2022
@Hotell Hotell deleted the hotell/build/migrate-v9-to-rolluped-dts-p4 branch May 5, 2022 10:25
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
…utton,conf-griffel,context-selector to ship rolluped only dts (microsoft#22823)

* feat(v9): migrate keyboard-keys,priority-overflow,alert,aria,avatar,button,conf-griffel,context-selector to ship rolluped only dts

* generate change files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants