-
Notifications
You must be signed in to change notification settings - Fork 2.9k
feat: replace lerna publish with nx release #30251
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
Merged
Hotell
merged 24 commits into
microsoft:master
from
Hotell:lerna/replace-lerna-publish-with-nx-release
Mar 1, 2024
Merged
feat: replace lerna publish with nx release #30251
Hotell
merged 24 commits into
microsoft:master
from
Hotell:lerna/replace-lerna-publish-with-nx-release
Mar 1, 2024
Conversation
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
9aaeaaa to
4f05194
Compare
Collaborator
📊 Bundle size reportUnchanged fixtures
|
Collaborator
Perf Analysis (
|
| Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
|---|---|---|---|---|---|
| Avatar | mount | 594 | 621 | 5000 | |
| Button | mount | 287 | 296 | 5000 | |
| Field | mount | 1154 | 1099 | 5000 | |
| FluentProvider | mount | 710 | 716 | 5000 | |
| FluentProviderWithTheme | mount | 91 | 82 | 10 | |
| FluentProviderWithTheme | virtual-rerender | 72 | 67 | 10 | |
| FluentProviderWithTheme | virtual-rerender-with-unmount | 78 | 85 | 10 | |
| MakeStyles | mount | 877 | 864 | 50000 | |
| Persona | mount | 1758 | 1705 | 5000 | |
| SpinButton | mount | 1384 | 1430 | 5000 |
Collaborator
Perf Analysis (
|
| Scenario | Current PR Ticks | Baseline Ticks | Ratio |
|---|---|---|---|
| PortalMinimalPerf.default | 95 | 81 | 1.17:1 |
| CardMinimalPerf.default | 328 | 293 | 1.12:1 |
| HeaderMinimalPerf.default | 224 | 208 | 1.08:1 |
| IconMinimalPerf.default | 395 | 367 | 1.08:1 |
| TreeWith60ListItems.default | 95 | 88 | 1.08:1 |
| ChatDuplicateMessagesPerf.default | 155 | 145 | 1.07:1 |
| RadioGroupMinimalPerf.default | 272 | 254 | 1.07:1 |
| ChatWithPopoverPerf.default | 191 | 180 | 1.06:1 |
| FlexMinimalPerf.default | 166 | 157 | 1.06:1 |
| ListMinimalPerf.default | 314 | 295 | 1.06:1 |
| ProviderMinimalPerf.default | 205 | 193 | 1.06:1 |
| SegmentMinimalPerf.default | 199 | 188 | 1.06:1 |
| TableManyItemsPerf.default | 1152 | 1089 | 1.06:1 |
| GridMinimalPerf.default | 192 | 182 | 1.05:1 |
| ListNestedPerf.default | 324 | 308 | 1.05:1 |
| AttachmentMinimalPerf.default | 85 | 82 | 1.04:1 |
| DividerMinimalPerf.default | 214 | 205 | 1.04:1 |
| HeaderSlotsPerf.default | 476 | 459 | 1.04:1 |
| ImageMinimalPerf.default | 236 | 228 | 1.04:1 |
| TableMinimalPerf.default | 233 | 225 | 1.04:1 |
| TextAreaMinimalPerf.default | 284 | 274 | 1.04:1 |
| AccordionMinimalPerf.default | 90 | 87 | 1.03:1 |
| AnimationMinimalPerf.default | 307 | 299 | 1.03:1 |
| DialogMinimalPerf.default | 452 | 438 | 1.03:1 |
| LabelMinimalPerf.default | 218 | 211 | 1.03:1 |
| PopupMinimalPerf.default | 348 | 337 | 1.03:1 |
| ProviderMergeThemesPerf.default | 651 | 633 | 1.03:1 |
| TextMinimalPerf.default | 200 | 194 | 1.03:1 |
| ButtonOverridesMissPerf.default | 635 | 624 | 1.02:1 |
| ChatMinimalPerf.default | 440 | 431 | 1.02:1 |
| CheckboxMinimalPerf.default | 1128 | 1107 | 1.02:1 |
| ItemLayoutMinimalPerf.default | 712 | 700 | 1.02:1 |
| LoaderMinimalPerf.default | 194 | 190 | 1.02:1 |
| MenuMinimalPerf.default | 503 | 494 | 1.02:1 |
| SkeletonMinimalPerf.default | 197 | 193 | 1.02:1 |
| ToolbarMinimalPerf.default | 538 | 526 | 1.02:1 |
| AttachmentSlotsPerf.default | 647 | 640 | 1.01:1 |
| CarouselMinimalPerf.default | 261 | 259 | 1.01:1 |
| DatepickerMinimalPerf.default | 3556 | 3517 | 1.01:1 |
| DropdownManyItemsPerf.default | 390 | 385 | 1.01:1 |
| SplitButtonMinimalPerf.default | 2227 | 2215 | 1.01:1 |
| CustomToolbarPrototype.default | 1469 | 1458 | 1.01:1 |
| ButtonSlotsPerf.default | 313 | 313 | 1:1 |
| EmbedMinimalPerf.default | 1874 | 1866 | 1:1 |
| FormMinimalPerf.default | 210 | 209 | 1:1 |
| MenuButtonMinimalPerf.default | 946 | 944 | 1:1 |
| RosterPerf.default | 1526 | 1530 | 1:1 |
| RefMinimalPerf.default | 106 | 106 | 1:1 |
| BoxMinimalPerf.default | 195 | 196 | 0.99:1 |
| TooltipMinimalPerf.default | 1279 | 1287 | 0.99:1 |
| TreeMinimalPerf.default | 466 | 472 | 0.99:1 |
| DropdownMinimalPerf.default | 1404 | 1429 | 0.98:1 |
| SliderMinimalPerf.default | 731 | 743 | 0.98:1 |
| VideoMinimalPerf.default | 466 | 476 | 0.98:1 |
| AvatarMinimalPerf.default | 110 | 113 | 0.97:1 |
| InputMinimalPerf.default | 531 | 549 | 0.97:1 |
| ButtonMinimalPerf.default | 85 | 89 | 0.96:1 |
| ReactionMinimalPerf.default | 201 | 210 | 0.96:1 |
| AlertMinimalPerf.default | 158 | 167 | 0.95:1 |
| StatusMinimalPerf.default | 378 | 397 | 0.95:1 |
| ListCommonPerf.default | 372 | 394 | 0.94:1 |
| ListWith60ListItems.default | 351 | 375 | 0.94:1 |
| LayoutMinimalPerf.default | 190 | 208 | 0.91:1 |
|
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. |
Collaborator
🕵 fluentuiv8 No visual regressions between this PR and main |
Collaborator
🕵 FluentUIV0 No visual regressions between this PR and main |
Collaborator
Perf Analysis (
|
| Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
|---|---|---|---|---|---|
| BaseButton | mount | 632 | 635 | 5000 | |
| Breadcrumb | mount | 1676 | 1666 | 1000 | |
| Checkbox | mount | 1701 | 1680 | 5000 | |
| CheckboxBase | mount | 1457 | 1486 | 5000 | |
| ChoiceGroup | mount | 2981 | 3000 | 5000 | |
| ComboBox | mount | 666 | 684 | 1000 | |
| CommandBar | mount | 6338 | 6386 | 1000 | |
| ContextualMenu | mount | 13652 | 12946 | 1000 | |
| DefaultButton | mount | 751 | 797 | 5000 | |
| DetailsRow | mount | 2254 | 2176 | 5000 | |
| DetailsRowFast | mount | 2245 | 2191 | 5000 | |
| DetailsRowNoStyles | mount | 2039 | 2007 | 5000 | |
| Dialog | mount | 2796 | 2751 | 1000 | |
| DocumentCardTitle | mount | 241 | 228 | 1000 | |
| Dropdown | mount | 1992 | 1964 | 5000 | |
| FocusTrapZone | mount | 1172 | 1209 | 5000 | |
| FocusZone | mount | 1112 | 1070 | 5000 | |
| GroupedList | mount | 41811 | 42079 | 2 | |
| GroupedList | virtual-rerender | 18008 | 20123 | 2 | |
| GroupedList | virtual-rerender-with-unmount | 52714 | 52559 | 2 | |
| GroupedListV2 | mount | 252 | 241 | 2 | |
| GroupedListV2 | virtual-rerender | 207 | 213 | 2 | |
| GroupedListV2 | virtual-rerender-with-unmount | 231 | 221 | 2 | |
| IconButton | mount | 1096 | 1075 | 5000 | |
| Label | mount | 331 | 331 | 5000 | |
| Layer | mount | 2768 | 2692 | 5000 | |
| Link | mount | 391 | 391 | 5000 | |
| MenuButton | mount | 923 | 944 | 5000 | |
| MessageBar | mount | 21726 | 21610 | 5000 | |
| Nav | mount | 1972 | 1969 | 1000 | |
| OverflowSet | mount | 787 | 788 | 5000 | |
| Panel | mount | 1772 | 1814 | 1000 | |
| Persona | mount | 752 | 761 | 1000 | |
| Pivot | mount | 853 | 867 | 1000 | |
| PrimaryButton | mount | 869 | 843 | 5000 | |
| Rating | mount | 4613 | 4686 | 5000 | |
| SearchBox | mount | 926 | 938 | 5000 | |
| Shimmer | mount | 1902 | 1908 | 5000 | |
| Slider | mount | 1345 | 1348 | 5000 | |
| SpinButton | mount | 2954 | 2898 | 5000 | |
| Spinner | mount | 391 | 395 | 5000 | |
| SplitButton | mount | 1870 | 1876 | 5000 | |
| Stack | mount | 415 | 409 | 5000 | |
| StackWithIntrinsicChildren | mount | 850 | 859 | 5000 | |
| StackWithTextChildren | mount | 2595 | 2631 | 5000 | |
| SwatchColorPicker | mount | 6199 | 6192 | 5000 | |
| TagPicker | mount | 1477 | 1462 | 5000 | |
| Text | mount | 367 | 368 | 5000 | |
| TextField | mount | 943 | 940 | 5000 | |
| ThemeProvider | mount | 823 | 821 | 5000 | |
| ThemeProvider | virtual-rerender | 590 | 577 | 5000 | |
| ThemeProvider | virtual-rerender-with-unmount | 1281 | 1264 | 5000 | |
| Toggle | mount | 601 | 611 | 5000 | |
| buttonNative | mount | 201 | 191 | 5000 |
Asset size changesSize Auditor did not detect a change in bundle size for any component! Baseline commit: 50176a643b2d47f9e45d6694293a7bc6dfac3dd7 (build) |
4f05194 to
dd42076
Compare
a4a1e9a to
376e001
Compare
3af1673 to
e3b5fab
Compare
JamesHenry
reviewed
Feb 1, 2024
JamesHenry
reviewed
Feb 1, 2024
f8d8101 to
db75779
Compare
Collaborator
🕵 fluentuiv9 No visual regressions between this PR and main |
a9e3448 to
ba8632f
Compare
ling1726
approved these changes
Feb 21, 2024
…er to remove lerna completely
…tAndFlushChanges from nx instead custom logic
…closures,runChange handles dryRun internally
…pi in order to resolve stagging issues
…sion' pipeline variable
…d assert during execution
66f178e to
ae46a99
Compare
5 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
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
react-northstar
New Behavior
nx releasenorthstar-releasenorthstar-releasenx releasenorthstarrelease groupnx release changelogis not being used because N* changelog is maintained by hand.versioncommand will update the changelog with new version though for smoother automation.cmd
versioncmd
publishUpdated release flow
do-not-delete-this-branch/0.xx.xyarn northstar:release version patch|minormasteryarn northstar:release publish@fluentui/react-northstar_v{0.xx.xx}Demo
version
Bump version to prepare release (
patch)yarn northstar:release version patchpublish
publish (re-run)
If something went wrong on CI (npm registry errors, or something else), we can re-run pipeline.
Related Issue(s)
nx release: provide configuration option to emit version ranges onversionexecution nrwl/nx#21044nx release: supportprojectsconfiguration filtering with similar patterns asrun-manynrwl/nx#21078nx release:publish -gtriggers publish for whole dependency tree beyond version group nrwl/nx#21116nx release: provide git tagging option forpublishcommand nrwl/nx#21118nx release:versioncommand should allow for only updating dependency versions when not already in range nrwl/nx#21457nx release: allow for version to not update yarn classic lock files by default nrwl/nx#21464nx release: provide schema forNxReleaseVersionConfigurationto improve DX nrwl/nx#21458nx release:versioncommand should bump dependencies outside group nrwl/nx#21466Follows: