Skip to content

Conversation

@Hotell
Copy link
Contributor

@Hotell Hotell commented Jan 9, 2024

Previous Behavior

New Behavior

see PR title

Related Issue(s)

@github-actions github-actions bot added this to the January Project Cycle Q1 2024 milestone Jan 9, 2024
@Hotell Hotell changed the title Lerna/remove lerna from northstar v3 feat(scripts-gulp): replace lerna with nx Jan 9, 2024
@Hotell Hotell force-pushed the lerna/remove-lerna-from-northstar-v3 branch from a12844a to e78ba16 Compare January 9, 2024 15:51
@Hotell Hotell marked this pull request as ready for review January 9, 2024 15:52
@Hotell Hotell requested review from a team as code owners January 9, 2024 15:52
@fabricteam
Copy link
Collaborator

fabricteam commented Jan 9, 2024

Perf Analysis (@fluentui/react-northstar)

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
AccordionMinimalPerf.default 82 70 1.17:1
LoaderMinimalPerf.default 196 174 1.13:1
PortalMinimalPerf.default 90 80 1.13:1
SkeletonMinimalPerf.default 204 188 1.09:1
AvatarMinimalPerf.default 114 106 1.08:1
ListMinimalPerf.default 307 290 1.06:1
RefMinimalPerf.default 116 109 1.06:1
TextAreaMinimalPerf.default 299 282 1.06:1
CheckboxMinimalPerf.default 1169 1115 1.05:1
HeaderSlotsPerf.default 475 453 1.05:1
InputMinimalPerf.default 554 529 1.05:1
ButtonSlotsPerf.default 343 331 1.04:1
DividerMinimalPerf.default 200 192 1.04:1
FormMinimalPerf.default 221 213 1.04:1
RadioGroupMinimalPerf.default 266 255 1.04:1
IconMinimalPerf.default 393 383 1.03:1
TreeMinimalPerf.default 479 463 1.03:1
FlexMinimalPerf.default 155 152 1.02:1
ListCommonPerf.default 397 391 1.02:1
MenuButtonMinimalPerf.default 982 963 1.02:1
ProviderMinimalPerf.default 202 198 1.02:1
StatusMinimalPerf.default 396 387 1.02:1
AttachmentMinimalPerf.default 81 80 1.01:1
AttachmentSlotsPerf.default 643 638 1.01:1
BoxMinimalPerf.default 193 191 1.01:1
ButtonOverridesMissPerf.default 653 645 1.01:1
CarouselMinimalPerf.default 266 263 1.01:1
ChatDuplicateMessagesPerf.default 151 150 1.01:1
DialogMinimalPerf.default 438 433 1.01:1
EmbedMinimalPerf.default 1862 1837 1.01:1
GridMinimalPerf.default 190 188 1.01:1
LabelMinimalPerf.default 217 215 1.01:1
PopupMinimalPerf.default 353 351 1.01:1
SliderMinimalPerf.default 742 737 1.01:1
ChatMinimalPerf.default 421 419 1:1
DatepickerMinimalPerf.default 3587 3587 1:1
DropdownMinimalPerf.default 1422 1424 1:1
HeaderMinimalPerf.default 210 210 1:1
ItemLayoutMinimalPerf.default 701 703 1:1
MenuMinimalPerf.default 495 494 1:1
SplitButtonMinimalPerf.default 2253 2261 1:1
TableMinimalPerf.default 226 225 1:1
CustomToolbarPrototype.default 1485 1480 1:1
VideoMinimalPerf.default 441 441 1:1
DropdownManyItemsPerf.default 388 390 0.99:1
ImageMinimalPerf.default 213 216 0.99:1
RosterPerf.default 1580 1598 0.99:1
ProviderMergeThemesPerf.default 649 654 0.99:1
TreeWith60ListItems.default 87 88 0.99:1
AlertMinimalPerf.default 153 156 0.98:1
CardMinimalPerf.default 306 312 0.98:1
ChatWithPopoverPerf.default 198 202 0.98:1
LayoutMinimalPerf.default 193 196 0.98:1
ListNestedPerf.default 316 321 0.98:1
ListWith60ListItems.default 357 363 0.98:1
ReactionMinimalPerf.default 210 214 0.98:1
TableManyItemsPerf.default 1113 1139 0.98:1
TextMinimalPerf.default 196 200 0.98:1
SegmentMinimalPerf.default 192 198 0.97:1
ToolbarMinimalPerf.default 526 542 0.97:1
ButtonMinimalPerf.default 88 92 0.96:1
TooltipMinimalPerf.default 1250 1299 0.96:1
AnimationMinimalPerf.default 287 301 0.95:1

@fabricteam
Copy link
Collaborator

fabricteam commented Jan 9, 2024

📊 Bundle size report

🤖 This report was generated against 702a25813f9be17fcda3c6c4d836c9beb28c14ce

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 9, 2024

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 76e8df0:

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

@fabricteam
Copy link
Collaborator

fabricteam commented Jan 9, 2024

🕵 FluentUIV0 No visual regressions between this PR and main

@Hotell Hotell force-pushed the lerna/remove-lerna-from-northstar-v3 branch from e78ba16 to 76e8df0 Compare January 9, 2024 16:48
@Hotell Hotell enabled auto-merge (squash) January 9, 2024 17:05
@size-auditor
Copy link

size-auditor bot commented Jan 9, 2024

Asset size changes

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

Baseline commit: 996d8fe7090f0667cb3308f320e596a476f82cbb (build)

@Hotell Hotell merged commit ab88e52 into microsoft:master Jan 9, 2024
@Hotell Hotell deleted the lerna/remove-lerna-from-northstar-v3 branch January 9, 2024 17:16
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jan 10, 2024
* master: (166 commits)
  Remove v0 dependency from v0 compat package (microsoft#30276)
  applying package updates
  Disallow `window` and `document` access for `@fluentui/react` and related packages. (microsoft#30063)
  Update Rating api and stories (microsoft#30092)
  TeachingPopover: Minor style changes (microsoft#30270)
  feat(scripts-gulp): replace lerna with nx (microsoft#30266)
  ci: remove canary and nightly functionality from northstar (microsoft#30264)
  List: Re-initialize on mount in React 18. (microsoft#29881)
  feat(scripts-monorepo): replace lerna/utils with pure nx apis (microsoft#30178)
  chore: remove react-timepicker-compat-preview (microsoft#30263)
  applying package updates
  feat(TimePicker-compat): stable release  (microsoft#30217)
  feat: Implement onPositioningEnd callback (microsoft#30177)
  applying package updates
  v8 registerIcons compat (microsoft#30003)
  Adding Planner, ToDoItem and updated Project filetype icons. Updating FabricCDN url to latest datecode. (microsoft#30079)
  Scaffolds more Nav components (microsoft#30227)
  chore: migrate to nx 17.2 (microsoft#30187)
  applying package updates
  feat: Update position when target or container dimensions change (microsoft#30179)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants