chore: migrate to ts v5.0.4#31898
Merged
Merged
Conversation
Collaborator
📊 Bundle size reportUnchanged fixtures
|
fabricteam
reviewed
Jul 2, 2024
Collaborator
Perf Analysis (
|
| Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
|---|---|---|---|---|---|
| Avatar | mount | 620 | 655 | 5000 | |
| Button | mount | 308 | 304 | 5000 | |
| Field | mount | 1123 | 1181 | 5000 | |
| FluentProvider | mount | 703 | 717 | 5000 | |
| FluentProviderWithTheme | mount | 78 | 85 | 10 | |
| FluentProviderWithTheme | virtual-rerender | 36 | 43 | 10 | |
| FluentProviderWithTheme | virtual-rerender-with-unmount | 77 | 73 | 10 | |
| MakeStyles | mount | 876 | 866 | 50000 | |
| Persona | mount | 1777 | 1752 | 5000 | |
| SpinButton | mount | 1352 | 1397 | 5000 | |
| SwatchPicker | mount | 1647 | 1668 | 5000 |
Collaborator
Perf Analysis (
|
| Scenario | Current PR Ticks | Baseline Ticks | Ratio | Regression Analysis |
|---|---|---|---|---|
| AlertMinimalPerf.default | 166 | 152 | 1.09:1 | analysis |
| PortalMinimalPerf.default | 88 | 82 | 1.07:1 | analysis |
| AttachmentMinimalPerf.default | 81 | 78 | 1.04:1 | analysis |
Perf tests with no regressions
| Scenario | Current PR Ticks | Baseline Ticks | Ratio |
|---|---|---|---|
| TreeWith60ListItems.default | 92 | 81 | 1.14:1 |
| ChatWithPopoverPerf.default | 215 | 198 | 1.09:1 |
| LoaderMinimalPerf.default | 203 | 186 | 1.09:1 |
| AvatarMinimalPerf.default | 114 | 107 | 1.07:1 |
| HeaderMinimalPerf.default | 217 | 204 | 1.06:1 |
| RefMinimalPerf.default | 118 | 111 | 1.06:1 |
| ChatDuplicateMessagesPerf.default | 150 | 143 | 1.05:1 |
| ButtonOverridesMissPerf.default | 687 | 663 | 1.04:1 |
| DividerMinimalPerf.default | 211 | 203 | 1.04:1 |
| FormMinimalPerf.default | 224 | 215 | 1.04:1 |
| PopupMinimalPerf.default | 357 | 344 | 1.04:1 |
| ProviderMinimalPerf.default | 207 | 199 | 1.04:1 |
| ToolbarMinimalPerf.default | 547 | 527 | 1.04:1 |
| AttachmentSlotsPerf.default | 643 | 624 | 1.03:1 |
| ChatMinimalPerf.default | 450 | 438 | 1.03:1 |
| FlexMinimalPerf.default | 164 | 159 | 1.03:1 |
| ListNestedPerf.default | 327 | 316 | 1.03:1 |
| MenuMinimalPerf.default | 509 | 495 | 1.03:1 |
| MenuButtonMinimalPerf.default | 952 | 925 | 1.03:1 |
| SegmentMinimalPerf.default | 204 | 199 | 1.03:1 |
| VideoMinimalPerf.default | 444 | 433 | 1.03:1 |
| DropdownMinimalPerf.default | 1433 | 1403 | 1.02:1 |
| ItemLayoutMinimalPerf.default | 693 | 678 | 1.02:1 |
| RadioGroupMinimalPerf.default | 255 | 250 | 1.02:1 |
| TreeMinimalPerf.default | 485 | 477 | 1.02:1 |
| AccordionMinimalPerf.default | 76 | 75 | 1.01:1 |
| AnimationMinimalPerf.default | 301 | 297 | 1.01:1 |
| BoxMinimalPerf.default | 191 | 190 | 1.01:1 |
| CarouselMinimalPerf.default | 257 | 254 | 1.01:1 |
| DialogMinimalPerf.default | 442 | 437 | 1.01:1 |
| LabelMinimalPerf.default | 222 | 220 | 1.01:1 |
| SkeletonMinimalPerf.default | 197 | 196 | 1.01:1 |
| TableManyItemsPerf.default | 1107 | 1092 | 1.01:1 |
| TooltipMinimalPerf.default | 1262 | 1255 | 1.01:1 |
| CheckboxMinimalPerf.default | 1123 | 1119 | 1:1 |
| DatepickerMinimalPerf.default | 3571 | 3577 | 1:1 |
| DropdownManyItemsPerf.default | 384 | 385 | 1:1 |
| ListCommonPerf.default | 382 | 382 | 1:1 |
| ProviderMergeThemesPerf.default | 657 | 656 | 1:1 |
| TextAreaMinimalPerf.default | 277 | 276 | 1:1 |
| CardMinimalPerf.default | 300 | 304 | 0.99:1 |
| EmbedMinimalPerf.default | 1896 | 1916 | 0.99:1 |
| InputMinimalPerf.default | 522 | 529 | 0.99:1 |
| LayoutMinimalPerf.default | 188 | 189 | 0.99:1 |
| SliderMinimalPerf.default | 739 | 746 | 0.99:1 |
| SplitButtonMinimalPerf.default | 2208 | 2223 | 0.99:1 |
| StatusMinimalPerf.default | 389 | 391 | 0.99:1 |
| TableMinimalPerf.default | 235 | 238 | 0.99:1 |
| TextMinimalPerf.default | 194 | 195 | 0.99:1 |
| GridMinimalPerf.default | 186 | 190 | 0.98:1 |
| IconMinimalPerf.default | 381 | 390 | 0.98:1 |
| CustomToolbarPrototype.default | 1448 | 1481 | 0.98:1 |
| ButtonSlotsPerf.default | 296 | 304 | 0.97:1 |
| RosterPerf.default | 1509 | 1560 | 0.97:1 |
| ListMinimalPerf.default | 297 | 310 | 0.96:1 |
| HeaderSlotsPerf.default | 447 | 470 | 0.95:1 |
| ImageMinimalPerf.default | 222 | 234 | 0.95:1 |
| ButtonMinimalPerf.default | 83 | 88 | 0.94:1 |
| ReactionMinimalPerf.default | 210 | 223 | 0.94:1 |
| ListWith60ListItems.default | 350 | 378 | 0.93:1 |
Collaborator
🕵 FluentUIV0 No visual regressions between this PR and main |
mainframev
commented
Jul 2, 2024
Collaborator
Perf Analysis (
|
| Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
|---|---|---|---|---|---|
| BaseButton | mount | 632 | 647 | 5000 | |
| Breadcrumb | mount | 1708 | 1708 | 1000 | |
| Checkbox | mount | 1714 | 1708 | 5000 | |
| CheckboxBase | mount | 1467 | 1548 | 5000 | |
| ChoiceGroup | mount | 2960 | 2937 | 5000 | |
| ComboBox | mount | 679 | 713 | 1000 | |
| CommandBar | mount | 6803 | 6707 | 1000 | |
| ContextualMenu | mount | err | 15879 | 1000 | |
| DefaultButton | mount | 802 | 821 | 5000 | |
| DetailsRow | mount | 2222 | 2203 | 5000 | |
| DetailsRowFast | mount | 2218 | 2254 | 5000 | |
| DetailsRowNoStyles | mount | 2065 | 2037 | 5000 | |
| Dialog | mount | 2806 | 2735 | 1000 | |
| DocumentCardTitle | mount | 230 | 239 | 1000 | |
| Dropdown | mount | 2005 | 2005 | 5000 | |
| FocusTrapZone | mount | 1169 | 1139 | 5000 | |
| FocusZone | mount | 1079 | 1068 | 5000 | |
| GroupedList | mount | 38283 | 42763 | 2 | |
| GroupedList | virtual-rerender | 20424 | 20499 | 2 | |
| GroupedList | virtual-rerender-with-unmount | 51765 | 51866 | 2 | |
| GroupedListV2 | mount | 240 | 235 | 2 | |
| GroupedListV2 | virtual-rerender | 212 | 210 | 2 | |
| GroupedListV2 | virtual-rerender-with-unmount | 226 | 238 | 2 | |
| IconButton | mount | 1131 | 1154 | 5000 | |
| Label | mount | 330 | 341 | 5000 | |
| Layer | mount | 2769 | 2774 | 5000 | |
| Link | mount | 401 | 395 | 5000 | |
| MenuButton | mount | 976 | 988 | 5000 | |
| MessageBar | mount | 21373 | 21383 | 5000 | |
| Nav | mount | 2044 | 2099 | 1000 | |
| OverflowSet | mount | 810 | 769 | 5000 | |
| Panel | mount | 1844 | 1804 | 1000 | |
| Persona | mount | 763 | 756 | 1000 | |
| Pivot | mount | 885 | 908 | 1000 | |
| PrimaryButton | mount | 959 | 940 | 5000 | |
| Rating | mount | 4710 | 4698 | 5000 | |
| SearchBox | mount | 917 | 913 | 5000 | |
| Shimmer | mount | 1872 | 1897 | 5000 | |
| Slider | mount | 1360 | 1358 | 5000 | |
| SpinButton | mount | 2988 | 3032 | 5000 | |
| Spinner | mount | 406 | 391 | 5000 | |
| SplitButton | mount | 1870 | 1898 | 5000 | |
| Stack | mount | 433 | 423 | 5000 | |
| StackWithIntrinsicChildren | mount | 864 | 885 | 5000 | |
| StackWithTextChildren | mount | 2771 | 2829 | 5000 | |
| SwatchColorPicker | mount | 6411 | 6385 | 5000 | |
| TagPicker | mount | 1508 | 1464 | 5000 | |
| Text | mount | 381 | 388 | 5000 | |
| TextField | mount | 971 | 964 | 5000 | |
| ThemeProvider | mount | 849 | 850 | 5000 | |
| ThemeProvider | virtual-rerender | 592 | 585 | 5000 | |
| ThemeProvider | virtual-rerender-with-unmount | 1295 | 1279 | 5000 | |
| Toggle | mount | 613 | 601 | 5000 | |
| buttonNative | mount | 199 | 198 | 5000 |
mainframev
force-pushed
the
chore/migrate-to-ts-5
branch
from
July 2, 2024 12:55
67ba4cb to
9b4daab
Compare
mainframev
commented
Jul 3, 2024
mainframev
force-pushed
the
chore/migrate-to-ts-5
branch
2 times, most recently
from
July 4, 2024 08:55
502765f to
33b8d2b
Compare
fabricteam
reviewed
Jul 4, 2024
mainframev
force-pushed
the
chore/migrate-to-ts-5
branch
4 times, most recently
from
July 4, 2024 14:08
69a0a32 to
f16213d
Compare
mainframev
commented
Jul 4, 2024
mainframev
commented
Jul 4, 2024
Collaborator
🕵 fluentuiv8 No visual regressions between this PR and main |
mainframev
marked this pull request as ready for review
July 8, 2024 13:02
Hotell
reviewed
Jul 8, 2024
mainframev
force-pushed
the
chore/migrate-to-ts-5
branch
2 times, most recently
from
July 8, 2024 14:09
50d4e41 to
ef93c9e
Compare
layershifter
approved these changes
Jul 8, 2024
janechu
approved these changes
Jul 8, 2024
mainframev
force-pushed
the
chore/migrate-to-ts-5
branch
from
July 9, 2024 13:00
ef93c9e to
54aa260
Compare
mainframev
force-pushed
the
chore/migrate-to-ts-5
branch
from
July 10, 2024 09:49
54aa260 to
ab25c3a
Compare
TristanWatanabe
approved these changes
Jul 10, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Previous Behavior
release-note v 4.9
New Behavior
Typescript migration to v5.
release-note v 5.0
Build and type-check speed:
v8 bundle size is affected:
DetailsListPickersThe increase is very minimal in total, about 445 bytes
forceConsistenCasingInFileNamesoption, as it's a default since v5.experimentalDecoratorsas it will continue to exist for the foreseeable future and better will be to solve it later separately.Related Issue(s)
@tsconfig/node20, mentioned in chore: bump @types/node types to v20 and @tsconfig to v18 #31510