-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: migrate to ts 5.2.2 #32859
chore: migrate to ts 5.2.2 #32859
Conversation
🕵 fluentui-web-components-v3 No visual regressions between this PR and main |
🕵 FluentUIV0 No visual regressions between this PR and main |
Perf Analysis (
|
Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
---|---|---|---|---|---|
Avatar | mount | 625 | 636 | 5000 | |
Button | mount | 313 | 301 | 5000 | |
Field | mount | 1122 | 1135 | 5000 | |
FluentProvider | mount | 710 | 706 | 5000 | |
FluentProviderWithTheme | mount | 83 | 91 | 10 | |
FluentProviderWithTheme | virtual-rerender | 37 | 35 | 10 | |
FluentProviderWithTheme | virtual-rerender-with-unmount | 84 | 86 | 10 | |
MakeStyles | mount | 887 | 874 | 50000 | |
Persona | mount | 1798 | 1713 | 5000 | |
SpinButton | mount | 1344 | 1442 | 5000 | |
SwatchPicker | mount | 1698 | 1665 | 5000 |
📊 Bundle size reportUnchanged fixtures
|
Perf Analysis (
|
Scenario | Current PR Ticks | Baseline Ticks | Ratio |
---|---|---|---|
PortalMinimalPerf.default | 102 | 83 | 1.23:1 |
AttachmentMinimalPerf.default | 99 | 82 | 1.21:1 |
AvatarMinimalPerf.default | 119 | 106 | 1.12:1 |
ButtonSlotsPerf.default | 325 | 291 | 1.12:1 |
TreeWith60ListItems.default | 96 | 87 | 1.1:1 |
LoaderMinimalPerf.default | 202 | 185 | 1.09:1 |
ImageMinimalPerf.default | 241 | 224 | 1.08:1 |
ProviderMinimalPerf.default | 214 | 199 | 1.08:1 |
AnimationMinimalPerf.default | 317 | 296 | 1.07:1 |
HeaderMinimalPerf.default | 220 | 205 | 1.07:1 |
IconMinimalPerf.default | 406 | 381 | 1.07:1 |
TreeMinimalPerf.default | 508 | 476 | 1.07:1 |
VideoMinimalPerf.default | 469 | 438 | 1.07:1 |
AccordionMinimalPerf.default | 93 | 88 | 1.06:1 |
CardMinimalPerf.default | 319 | 300 | 1.06:1 |
RefMinimalPerf.default | 119 | 112 | 1.06:1 |
TableMinimalPerf.default | 249 | 234 | 1.06:1 |
AttachmentSlotsPerf.default | 654 | 620 | 1.05:1 |
RadioGroupMinimalPerf.default | 274 | 262 | 1.05:1 |
SkeletonMinimalPerf.default | 206 | 197 | 1.05:1 |
BoxMinimalPerf.default | 207 | 199 | 1.04:1 |
ReactionMinimalPerf.default | 216 | 208 | 1.04:1 |
CarouselMinimalPerf.default | 267 | 259 | 1.03:1 |
FormMinimalPerf.default | 228 | 222 | 1.03:1 |
InputMinimalPerf.default | 554 | 539 | 1.03:1 |
ListNestedPerf.default | 337 | 326 | 1.03:1 |
CustomToolbarPrototype.default | 1515 | 1467 | 1.03:1 |
CheckboxMinimalPerf.default | 1177 | 1159 | 1.02:1 |
LabelMinimalPerf.default | 225 | 220 | 1.02:1 |
ListCommonPerf.default | 407 | 398 | 1.02:1 |
MenuMinimalPerf.default | 521 | 509 | 1.02:1 |
PopupMinimalPerf.default | 351 | 345 | 1.02:1 |
ProviderMergeThemesPerf.default | 665 | 655 | 1.02:1 |
SliderMinimalPerf.default | 746 | 730 | 1.02:1 |
TextMinimalPerf.default | 206 | 202 | 1.02:1 |
ChatMinimalPerf.default | 437 | 434 | 1.01:1 |
DividerMinimalPerf.default | 212 | 210 | 1.01:1 |
DropdownManyItemsPerf.default | 400 | 398 | 1.01:1 |
GridMinimalPerf.default | 192 | 190 | 1.01:1 |
HeaderSlotsPerf.default | 474 | 470 | 1.01:1 |
ListMinimalPerf.default | 314 | 310 | 1.01:1 |
ToolbarMinimalPerf.default | 539 | 533 | 1.01:1 |
AlertMinimalPerf.default | 167 | 167 | 1:1 |
DatepickerMinimalPerf.default | 3719 | 3704 | 1:1 |
FlexMinimalPerf.default | 153 | 153 | 1:1 |
LayoutMinimalPerf.default | 203 | 204 | 1:1 |
RosterPerf.default | 1554 | 1555 | 1:1 |
TableManyItemsPerf.default | 1139 | 1136 | 1:1 |
TooltipMinimalPerf.default | 1281 | 1277 | 1:1 |
DropdownMinimalPerf.default | 1429 | 1450 | 0.99:1 |
EmbedMinimalPerf.default | 1863 | 1890 | 0.99:1 |
ItemLayoutMinimalPerf.default | 708 | 716 | 0.99:1 |
SplitButtonMinimalPerf.default | 2265 | 2298 | 0.99:1 |
TextAreaMinimalPerf.default | 289 | 291 | 0.99:1 |
ButtonMinimalPerf.default | 88 | 90 | 0.98:1 |
DialogMinimalPerf.default | 439 | 447 | 0.98:1 |
StatusMinimalPerf.default | 402 | 412 | 0.98:1 |
ChatDuplicateMessagesPerf.default | 149 | 154 | 0.97:1 |
ListWith60ListItems.default | 368 | 380 | 0.97:1 |
ButtonOverridesMissPerf.default | 640 | 669 | 0.96:1 |
MenuButtonMinimalPerf.default | 957 | 1012 | 0.95:1 |
SegmentMinimalPerf.default | 197 | 207 | 0.95:1 |
ChatWithPopoverPerf.default | 197 | 210 | 0.94:1 |
0de3441
to
ce11185
Compare
Perf Analysis (
|
Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
---|---|---|---|---|---|
BaseButton | mount | 615 | 638 | 5000 | |
Breadcrumb | mount | 1692 | 1654 | 1000 | |
Checkbox | mount | 1676 | 1691 | 5000 | |
CheckboxBase | mount | 1456 | 1482 | 5000 | |
ChoiceGroup | mount | 2969 | 2999 | 5000 | |
ComboBox | mount | 686 | 659 | 1000 | |
CommandBar | mount | 6505 | 6521 | 1000 | |
ContextualMenu | mount | 11699 | 12129 | 1000 | |
DefaultButton | mount | 796 | 793 | 5000 | |
DetailsRow | mount | 2150 | 2184 | 5000 | |
DetailsRowFast | mount | 2243 | 2213 | 5000 | |
DetailsRowNoStyles | mount | 2021 | 2041 | 5000 | |
Dialog | mount | 2779 | 2806 | 1000 | |
DocumentCardTitle | mount | 239 | 239 | 1000 | |
Dropdown | mount | 2026 | 2004 | 5000 | |
FocusTrapZone | mount | 1159 | 1154 | 5000 | |
FocusZone | mount | 1115 | 1052 | 5000 | |
GroupedList | mount | 42537 | 42307 | 2 | |
GroupedList | virtual-rerender | 20259 | 20239 | 2 | |
GroupedList | virtual-rerender-with-unmount | 51713 | 52021 | 2 | |
GroupedListV2 | mount | 243 | 228 | 2 | |
GroupedListV2 | virtual-rerender | 217 | 223 | 2 | |
GroupedListV2 | virtual-rerender-with-unmount | 235 | 227 | 2 | |
IconButton | mount | 1146 | 1130 | 5000 | |
Label | mount | 349 | 348 | 5000 | |
Layer | mount | 2746 | 2724 | 5000 | |
Link | mount | 387 | 384 | 5000 | |
MenuButton | mount | 990 | 983 | 5000 | |
MessageBar | mount | 21357 | 21302 | 5000 | |
Nav | mount | 2053 | 2013 | 1000 | |
OverflowSet | mount | 796 | 777 | 5000 | |
Panel | mount | 1778 | 1853 | 1000 | |
Persona | mount | 730 | 763 | 1000 | |
Pivot | mount | 921 | 888 | 1000 | |
PrimaryButton | mount | 927 | 933 | 5000 | |
Rating | mount | 4606 | 4717 | 5000 | |
SearchBox | mount | 928 | 914 | 5000 | |
Shimmer | mount | 1923 | 1891 | 5000 | |
Slider | mount | 1325 | 1357 | 5000 | |
SpinButton | mount | 2965 | 2990 | 5000 | |
Spinner | mount | 388 | 399 | 5000 | |
SplitButton | mount | 1851 | 1844 | 5000 | |
Stack | mount | 425 | 424 | 5000 | |
StackWithIntrinsicChildren | mount | 868 | 874 | 5000 | |
StackWithTextChildren | mount | 2745 | 2767 | 5000 | |
SwatchColorPicker | mount | 6414 | 6355 | 5000 | |
TagPicker | mount | 1477 | 1473 | 5000 | |
Text | mount | 395 | 397 | 5000 | |
TextField | mount | 911 | 951 | 5000 | |
ThemeProvider | mount | 860 | 860 | 5000 | |
ThemeProvider | virtual-rerender | 584 | 579 | 5000 | |
ThemeProvider | virtual-rerender-with-unmount | 1290 | 1323 | 5000 | |
Toggle | mount | 596 | 614 | 5000 | |
buttonNative | mount | 193 | 197 | 5000 |
e8f410a
to
877529c
Compare
5b938cd
to
07f2011
Compare
07f2011
to
a4652bb
Compare
a4652bb
to
597fc44
Compare
I don't think we want to set |
9077cb5
to
a0fca0f
Compare
tools/workspace-plugin/src/generators/cypress-component-configuration/index.ts
Outdated
Show resolved
Hide resolved
tools/workspace-plugin/src/generators/cypress-component-configuration/index.ts
Outdated
Show resolved
Hide resolved
gentle ping @radium-v |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
Reviewing now! Sorry for the late response. |
@Hotell addressing your concerns:
Babel is just an underlying dependency for the web components Storybook instance - we don't use it directly in any of our processes. We'd have to redo the web components Storybook configuration in order to not use Babel. I mentioned before I'd prefer to switch our Storybook instance to use Vite, but that would have to be done as part of the migration to Storybook 8.
In order to use |
fe65b95
to
6426796
Compare
6d651cb
to
25abc7e
Compare
Pull request demo site: URL |
25abc7e
to
1dfe2b1
Compare
Release notes:
What's addressed:
${configDir}
is introduced.Previous Behavior
New Behavior
Related Issue(s)
A little slowdown should be caused by explicit type roots, once we will update to ts 5.5