-
Notifications
You must be signed in to change notification settings - Fork 2.9k
chore: remove executors that use lerna under the hood, improve/cleanup northstar release cli tool, cleanup global npm scripts #30202
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: remove executors that use lerna under the hood, improve/cleanup northstar release cli tool, cleanup global npm scripts #30202
Conversation
…hich are redundant and replaced by nx
|
Some questions:
Getting fluent packages What was the original intent of following logic? https://github.com/microsoft/fluentui/pull/30202/files#diff-8628480ad3670d357c3c8f31928e24bf4895ac31705fbf6cb3e6dd227e4b2e9eL121 ATM this will return all monorepo packages and thus will apply those nightly versions to all of them. Is this correct or it should process only northstar packages |
Perf Analysis (
|
| Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
|---|---|---|---|---|---|
| Avatar | mount | 633 | 639 | 5000 | |
| Button | mount | 306 | 312 | 5000 | |
| Field | mount | 1148 | 1155 | 5000 | |
| FluentProvider | mount | 707 | 703 | 5000 | |
| FluentProviderWithTheme | mount | 78 | 79 | 10 | |
| FluentProviderWithTheme | virtual-rerender | 66 | 65 | 10 | |
| FluentProviderWithTheme | virtual-rerender-with-unmount | 70 | 75 | 10 | |
| MakeStyles | mount | 849 | 821 | 50000 | |
| Persona | mount | 1744 | 1706 | 5000 | |
| SpinButton | mount | 1397 | 1400 | 5000 |
|
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 b08b4dd:
|
🕵 fluentuiv8 No visual regressions between this PR and main |
Perf Analysis (
|
| Scenario | Current PR Ticks | Baseline Ticks | Ratio | Regression Analysis |
|---|---|---|---|---|
| AccordionMinimalPerf.default | 86 | 82 | 1.05:1 | analysis |
Perf tests with no regressions
| Scenario | Current PR Ticks | Baseline Ticks | Ratio |
|---|---|---|---|
| ChatDuplicateMessagesPerf.default | 162 | 141 | 1.15:1 |
| DividerMinimalPerf.default | 210 | 195 | 1.08:1 |
| FormMinimalPerf.default | 230 | 213 | 1.08:1 |
| ReactionMinimalPerf.default | 221 | 205 | 1.08:1 |
| RefMinimalPerf.default | 117 | 110 | 1.06:1 |
| AlertMinimalPerf.default | 160 | 153 | 1.05:1 |
| ChatWithPopoverPerf.default | 203 | 195 | 1.04:1 |
| HeaderMinimalPerf.default | 212 | 203 | 1.04:1 |
| RadioGroupMinimalPerf.default | 260 | 251 | 1.04:1 |
| TableMinimalPerf.default | 232 | 223 | 1.04:1 |
| TextMinimalPerf.default | 192 | 184 | 1.04:1 |
| BoxMinimalPerf.default | 198 | 193 | 1.03:1 |
| ButtonOverridesMissPerf.default | 666 | 647 | 1.03:1 |
| MenuMinimalPerf.default | 510 | 497 | 1.03:1 |
| MenuButtonMinimalPerf.default | 964 | 936 | 1.03:1 |
| CheckboxMinimalPerf.default | 1165 | 1137 | 1.02:1 |
| FlexMinimalPerf.default | 156 | 153 | 1.02:1 |
| HeaderSlotsPerf.default | 467 | 458 | 1.02:1 |
| RosterPerf.default | 1605 | 1566 | 1.02:1 |
| SegmentMinimalPerf.default | 194 | 191 | 1.02:1 |
| VideoMinimalPerf.default | 442 | 435 | 1.02:1 |
| ButtonSlotsPerf.default | 307 | 303 | 1.01:1 |
| CarouselMinimalPerf.default | 259 | 257 | 1.01:1 |
| DialogMinimalPerf.default | 451 | 446 | 1.01:1 |
| GridMinimalPerf.default | 190 | 188 | 1.01:1 |
| LabelMinimalPerf.default | 222 | 220 | 1.01:1 |
| LayoutMinimalPerf.default | 198 | 197 | 1.01:1 |
| ListMinimalPerf.default | 300 | 298 | 1.01:1 |
| PopupMinimalPerf.default | 354 | 351 | 1.01:1 |
| StatusMinimalPerf.default | 395 | 393 | 1.01:1 |
| AvatarMinimalPerf.default | 110 | 110 | 1:1 |
| CardMinimalPerf.default | 312 | 311 | 1:1 |
| ChatMinimalPerf.default | 425 | 424 | 1:1 |
| DatepickerMinimalPerf.default | 3507 | 3495 | 1:1 |
| EmbedMinimalPerf.default | 1883 | 1876 | 1:1 |
| ToolbarMinimalPerf.default | 539 | 540 | 1:1 |
| AttachmentSlotsPerf.default | 635 | 643 | 0.99:1 |
| DropdownMinimalPerf.default | 1416 | 1425 | 0.99:1 |
| ImageMinimalPerf.default | 223 | 226 | 0.99:1 |
| ItemLayoutMinimalPerf.default | 707 | 715 | 0.99:1 |
| LoaderMinimalPerf.default | 190 | 191 | 0.99:1 |
| ProviderMinimalPerf.default | 196 | 197 | 0.99:1 |
| SliderMinimalPerf.default | 735 | 739 | 0.99:1 |
| SplitButtonMinimalPerf.default | 2199 | 2221 | 0.99:1 |
| TableManyItemsPerf.default | 1095 | 1108 | 0.99:1 |
| TreeMinimalPerf.default | 477 | 484 | 0.99:1 |
| AnimationMinimalPerf.default | 290 | 296 | 0.98:1 |
| DropdownManyItemsPerf.default | 387 | 393 | 0.98:1 |
| PortalMinimalPerf.default | 86 | 88 | 0.98:1 |
| CustomToolbarPrototype.default | 1435 | 1466 | 0.98:1 |
| TooltipMinimalPerf.default | 1302 | 1333 | 0.98:1 |
| ListCommonPerf.default | 375 | 386 | 0.97:1 |
| TextAreaMinimalPerf.default | 292 | 302 | 0.97:1 |
| InputMinimalPerf.default | 518 | 539 | 0.96:1 |
| ListNestedPerf.default | 319 | 334 | 0.96:1 |
| ProviderMergeThemesPerf.default | 633 | 663 | 0.95:1 |
| TreeWith60ListItems.default | 75 | 79 | 0.95:1 |
| SkeletonMinimalPerf.default | 189 | 202 | 0.94:1 |
| AttachmentMinimalPerf.default | 80 | 86 | 0.93:1 |
| IconMinimalPerf.default | 381 | 408 | 0.93:1 |
| ListWith60ListItems.default | 341 | 373 | 0.91:1 |
| ButtonMinimalPerf.default | 85 | 99 | 0.86:1 |
🕵 FluentUIV0 No visual regressions between this PR and main |
📊 Bundle size reportUnchanged fixtures
|
Asset size changesSize Auditor did not detect a change in bundle size for any component! Baseline commit: 2a42b52a62aa5bc6e2c0fb81e43d4edf0b0fc4e9 (build) |
🕵 fluentuiv9 No visual regressions between this PR and main |
Perf Analysis (
|
| Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
|---|---|---|---|---|---|
| BaseButton | mount | 625 | 627 | 5000 | |
| Breadcrumb | mount | 1667 | 1659 | 1000 | |
| Checkbox | mount | 1675 | 1665 | 5000 | |
| CheckboxBase | mount | 1502 | 1471 | 5000 | |
| ChoiceGroup | mount | 2969 | 2918 | 5000 | |
| ComboBox | mount | 647 | 657 | 1000 | |
| CommandBar | mount | 6151 | 6131 | 1000 | |
| ContextualMenu | mount | 13509 | 13559 | 1000 | |
| DefaultButton | mount | 746 | 750 | 5000 | |
| DetailsRow | mount | 2159 | 2212 | 5000 | |
| DetailsRowFast | mount | 2188 | 2241 | 5000 | |
| DetailsRowNoStyles | mount | 2015 | 2012 | 5000 | |
| Dialog | mount | 2619 | 2813 | 1000 | |
| DocumentCardTitle | mount | 235 | 228 | 1000 | |
| Dropdown | mount | 1938 | 1948 | 5000 | |
| FocusTrapZone | mount | 1083 | 1123 | 5000 | |
| FocusZone | mount | 1065 | 1059 | 5000 | |
| GroupedList | mount | 41664 | 41882 | 2 | |
| GroupedList | virtual-rerender | 20260 | 20466 | 2 | |
| GroupedList | virtual-rerender-with-unmount | 50443 | 51018 | 2 | |
| GroupedListV2 | mount | 224 | 228 | 2 | |
| GroupedListV2 | virtual-rerender | 217 | 208 | 2 | |
| GroupedListV2 | virtual-rerender-with-unmount | 222 | 224 | 2 | |
| IconButton | mount | 1098 | 1088 | 5000 | |
| Label | mount | 334 | 343 | 5000 | |
| Layer | mount | 2713 | 2736 | 5000 | |
| Link | mount | 392 | 397 | 5000 | |
| MenuButton | mount | 946 | 955 | 5000 | |
| MessageBar | mount | 21357 | 21347 | 5000 | |
| Nav | mount | 1930 | 1978 | 1000 | |
| OverflowSet | mount | 784 | 761 | 5000 | |
| Panel | mount | 1781 | 1780 | 1000 | |
| Persona | mount | 702 | 753 | 1000 | |
| Pivot | mount | 854 | 872 | 1000 | |
| PrimaryButton | mount | 850 | 853 | 5000 | |
| Rating | mount | 4544 | 4593 | 5000 | |
| SearchBox | mount | 884 | 885 | 5000 | |
| Shimmer | mount | 1881 | 1865 | 5000 | |
| Slider | mount | 1290 | 1323 | 5000 | |
| SpinButton | mount | 2882 | 2805 | 5000 | |
| Spinner | mount | 383 | 375 | 5000 | |
| SplitButton | mount | 1840 | 1819 | 5000 | |
| Stack | mount | 407 | 419 | 5000 | |
| StackWithIntrinsicChildren | mount | 865 | 841 | 5000 | |
| StackWithTextChildren | mount | 2593 | 2568 | 5000 | |
| SwatchColorPicker | mount | 6196 | 6009 | 5000 | |
| TagPicker | mount | 1409 | 1451 | 5000 | |
| Text | mount | 378 | 373 | 5000 | |
| TextField | mount | 923 | 939 | 5000 | |
| ThemeProvider | mount | 807 | 809 | 5000 | |
| ThemeProvider | virtual-rerender | 590 | 588 | 5000 | |
| ThemeProvider | virtual-rerender-with-unmount | 1254 | 1267 | 5000 | |
| Toggle | mount | 607 | 614 | 5000 | |
| buttonNative | mount | 195 | 193 | 5000 |
@Hotell, the goal is to process all packages within packages/fluentui, not all packages in the monorepo. However I just gave it a try and this line 121 prints out all packages in the monorepo. This is probably why it hasn't been working since last April :/ |
gotcha. this will be addressed in follow-up PR ( also added "follow up" section in PR description. ty! |
…p northstar release cli tool, cleanup global npm scripts (microsoft#30202)
Previous Behavior
As we are moving towards nx (that is used under the hood by lerna) we wanna remove unnecessary infra code from our codebase. This is one of many PR's that will address this initiative.
New Behavior
various northstar related npm script aliases removed
northstar related global scripts aggegated under "northstar:*" prefix
fluent release script now works as proper CLI
custom scripts leveraging lerna are removed
Follow up (another PR):
Related Issue(s)