Skip to content

Conversation

@Hotell
Copy link
Contributor

@Hotell Hotell commented Jul 17, 2023

Previous Behavior

lighrail doesnt work with affected builds and needs everything build on every PR.

as we enabled since some time ago, every PR that wont build all projects or has some mixup state from previous PR merged into master get following confusing metrics:

image

New Behavior

see PR title

Related Issue(s)

…s behaviour and needs everything build on every PR
@github-actions github-actions bot added this to the July Project Cycle Q3 2023 milestone Jul 17, 2023
@Hotell Hotell added Area: Build System Fluent UI react-northstar (v0) Work related to Fluent UI V0 Fluent UI react (v8) Issues about @fluentui/react (v8) CI labels Jul 17, 2023
@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
InfoButton mount 13 11 5000 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 602 620 5000
Button mount 292 292 5000
Field mount 1071 1072 5000
FluentProvider mount 651 670 5000
FluentProviderWithTheme mount 69 77 10
FluentProviderWithTheme virtual-rerender 61 68 10
FluentProviderWithTheme virtual-rerender-with-unmount 75 69 10
InfoButton mount 13 11 5000 Possible regression
MakeStyles mount 847 833 50000
Persona mount 1699 1574 5000
SpinButton mount 1305 1284 5000

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-northstar)

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
AttachmentMinimalPerf.default 90 80 1.13:1
AlertMinimalPerf.default 161 147 1.1:1
ImageMinimalPerf.default 228 212 1.08:1
RefMinimalPerf.default 113 105 1.08:1
ButtonMinimalPerf.default 90 84 1.07:1
InputMinimalPerf.default 553 519 1.07:1
AnimationMinimalPerf.default 306 289 1.06:1
AvatarMinimalPerf.default 113 107 1.06:1
CarouselMinimalPerf.default 267 253 1.06:1
DropdownManyItemsPerf.default 391 370 1.06:1
PortalMinimalPerf.default 85 80 1.06:1
BoxMinimalPerf.default 198 188 1.05:1
StatusMinimalPerf.default 423 403 1.05:1
TreeWith60ListItems.default 90 86 1.05:1
RosterPerf.default 1544 1483 1.04:1
ButtonSlotsPerf.default 314 305 1.03:1
DropdownMinimalPerf.default 1428 1388 1.03:1
EmbedMinimalPerf.default 1910 1861 1.03:1
ListNestedPerf.default 328 319 1.03:1
MenuButtonMinimalPerf.default 974 950 1.03:1
TreeMinimalPerf.default 483 469 1.03:1
ChatDuplicateMessagesPerf.default 153 150 1.02:1
DatepickerMinimalPerf.default 3568 3510 1.02:1
FormMinimalPerf.default 226 222 1.02:1
LabelMinimalPerf.default 219 214 1.02:1
ReactionMinimalPerf.default 212 207 1.02:1
TableMinimalPerf.default 239 235 1.02:1
ToolbarMinimalPerf.default 540 529 1.02:1
ChatMinimalPerf.default 434 431 1.01:1
DialogMinimalPerf.default 444 438 1.01:1
DividerMinimalPerf.default 206 203 1.01:1
HeaderMinimalPerf.default 212 209 1.01:1
ItemLayoutMinimalPerf.default 715 708 1.01:1
ListMinimalPerf.default 303 300 1.01:1
MenuMinimalPerf.default 502 499 1.01:1
PopupMinimalPerf.default 349 345 1.01:1
TextMinimalPerf.default 198 196 1.01:1
ButtonOverridesMissPerf.default 650 647 1:1
HeaderSlotsPerf.default 463 461 1:1
ProviderMergeThemesPerf.default 670 667 1:1
RadioGroupMinimalPerf.default 262 261 1:1
SegmentMinimalPerf.default 196 196 1:1
TableManyItemsPerf.default 1133 1128 1:1
ChatWithPopoverPerf.default 194 195 0.99:1
CheckboxMinimalPerf.default 1123 1139 0.99:1
FlexMinimalPerf.default 149 151 0.99:1
ProviderMinimalPerf.default 197 200 0.99:1
SliderMinimalPerf.default 729 735 0.99:1
SplitButtonMinimalPerf.default 2248 2264 0.99:1
CustomToolbarPrototype.default 1462 1481 0.99:1
GridMinimalPerf.default 192 195 0.98:1
TextAreaMinimalPerf.default 286 293 0.98:1
TooltipMinimalPerf.default 1248 1272 0.98:1
AttachmentSlotsPerf.default 629 648 0.97:1
CardMinimalPerf.default 311 319 0.97:1
ListCommonPerf.default 381 394 0.97:1
SkeletonMinimalPerf.default 193 200 0.97:1
IconMinimalPerf.default 365 386 0.95:1
VideoMinimalPerf.default 413 435 0.95:1
ListWith60ListItems.default 362 384 0.94:1
LoaderMinimalPerf.default 182 193 0.94:1
AccordionMinimalPerf.default 78 84 0.93:1
LayoutMinimalPerf.default 192 206 0.93:1

@codesandbox-ci
Copy link

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 cf63434:

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

@fabricteam
Copy link
Collaborator

🕵 fluentuiv8 No visual regressions between this PR and main

@fabricteam
Copy link
Collaborator

🕵 FluentUIV0 No visual regressions between this PR and main

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
global-context
createContext
510 B
330 B
global-context
createContextSelector
537 B
342 B
priority-overflow
createOverflowManager
4.248 kB
1.671 kB
react-accordion
Accordion (including children components)
88.26 kB
26.718 kB
react-alert
Alert
82.565 kB
21.726 kB
react-avatar
Avatar
47.713 kB
14.513 kB
react-avatar
AvatarGroup
15.695 kB
6.314 kB
react-avatar
AvatarGroupItem
63.889 kB
18.993 kB
react-badge
Badge
23.604 kB
7.272 kB
react-badge
CounterBadge
24.506 kB
7.574 kB
react-badge
PresenceBadge
22.227 kB
7.881 kB
react-button
Button
36.792 kB
9.472 kB
react-button
CompoundButton
43.946 kB
10.952 kB
react-button
MenuButton
40.978 kB
10.669 kB
react-button
SplitButton
49.211 kB
12.232 kB
react-button
ToggleButton
55.074 kB
11.368 kB
react-card
Card - All
89.36 kB
25.284 kB
react-card
Card
83.778 kB
23.685 kB
react-card
CardFooter
9.24 kB
3.907 kB
react-card
CardHeader
11.652 kB
4.741 kB
react-card
CardPreview
10.048 kB
4.254 kB
react-checkbox
Checkbox
33.198 kB
10.718 kB
react-combobox
Combobox (including child components)
87.173 kB
28.125 kB
react-combobox
Dropdown (including child components)
85.576 kB
27.723 kB
react-components
react-components: Button, FluentProvider & webLightTheme
65.175 kB
17.915 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
210.68 kB
58.781 kB
react-components
react-components: FluentProvider & webLightTheme
36.409 kB
12.003 kB
react-datepicker-compat
DatePicker Compat
223.368 kB
59.265 kB
react-dialog
Dialog (including children components)
92.736 kB
27.579 kB
react-divider
Divider
17.49 kB
6.364 kB
react-field
Field
18.148 kB
6.97 kB
react-image
Image
11.561 kB
4.637 kB
react-infobutton
InfoButton
131.335 kB
40.311 kB
react-infobutton
InfoLabel
134.94 kB
41.443 kB
react-input
Input
24.232 kB
7.781 kB
react-label
Label
10.186 kB
4.253 kB
react-link
Link
12.782 kB
5.247 kB
react-menu
Menu (including children components)
132.673 kB
40.613 kB
react-menu
Menu (including selectable components)
135.437 kB
41.106 kB
react-overflow
hooks only
12.505 kB
4.626 kB
react-persona
Persona
55.267 kB
16.525 kB
react-popover
Popover
119.925 kB
36.895 kB
react-portal
Portal
11.935 kB
4.421 kB
react-portal-compat
PortalCompatProvider
6.473 kB
2.196 kB
react-positioning
usePositioning
24.272 kB
8.866 kB
react-progress
ProgressBar
13.941 kB
5.498 kB
react-provider
FluentProvider
18.129 kB
6.73 kB
react-radio
Radio
26.991 kB
8.626 kB
react-radio
RadioGroup
11.376 kB
4.763 kB
react-select
Select
24.929 kB
8.713 kB
react-slider
Slider
34.87 kB
11.178 kB
react-spinbutton
SpinButton
33.73 kB
10.337 kB
react-spinner
Spinner
21.376 kB
7.029 kB
react-switch
Switch
29.485 kB
9.228 kB
react-table
DataGrid
158.282 kB
43.276 kB
react-table
Table (Primitives only)
44.666 kB
12.442 kB
react-table
Table as DataGrid
132.365 kB
33.849 kB
react-table
Table (Selection only)
77.993 kB
19.245 kB
react-table
Table (Sort only)
76.978 kB
18.973 kB
react-tags-preview
InteractionTag
33.855 kB
9.08 kB
react-tags-preview
Tag
25.936 kB
8.483 kB
react-tags-preview
TagGroup
69.142 kB
20.294 kB
react-text
Text - Default
12.574 kB
4.982 kB
react-text
Text - Wrappers
15.726 kB
5.302 kB
react-textarea
Textarea
27.703 kB
9.126 kB
react-theme
Single theme token import
69 B
89 B
react-theme
Teams: all themes
31.987 kB
6.85 kB
react-theme
Teams: Light theme
18.121 kB
5.194 kB
react-toast
Toast (including Toaster)
78.365 kB
23.67 kB
react-tooltip
Tooltip
47.463 kB
16.655 kB
react-utilities
SSRProvider
180 B
159 B
🤖 This report was generated against c3f4b77f8160185518b1fea6d74be239519fba73

@fabricteam
Copy link
Collaborator

🕵 fluentuiv9 No visual regressions between this PR and main

@size-auditor
Copy link

size-auditor bot commented Jul 17, 2023

Asset size changes

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

Baseline commit: c3f4b77f8160185518b1fea6d74be239519fba73 (build)

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
BaseButton mount 627 628 5000
Breadcrumb mount 1678 1676 1000
Checkbox mount 1681 1695 5000
CheckboxBase mount 1456 1472 5000
ChoiceGroup mount 2937 2923 5000
ComboBox mount 665 647 1000
CommandBar mount 6245 6184 1000
ContextualMenu mount 13639 13360 1000
DefaultButton mount 741 745 5000
DetailsRow mount 2183 2206 5000
DetailsRowFast mount 2187 2173 5000
DetailsRowNoStyles mount 2014 1977 5000
Dialog mount 2739 2667 1000
DocumentCardTitle mount 233 237 1000
Dropdown mount 2031 2025 5000
FocusTrapZone mount 1143 1137 5000
FocusZone mount 1068 1085 5000
GroupedList mount 41259 41465 2
GroupedList virtual-rerender 19618 19818 2
GroupedList virtual-rerender-with-unmount 50695 49989 2
GroupedListV2 mount 219 222 2
GroupedListV2 virtual-rerender 212 211 2
GroupedListV2 virtual-rerender-with-unmount 231 230 2
IconButton mount 1081 1111 5000
Label mount 333 328 5000
Layer mount 2724 2739 5000
Link mount 391 388 5000
MenuButton mount 930 945 5000
MessageBar mount 21628 21480 5000
Nav mount 1948 1901 1000
OverflowSet mount 776 782 5000
Panel mount 1794 1797 1000
Persona mount 728 751 1000
Pivot mount 868 881 1000
PrimaryButton mount 849 842 5000
Rating mount 4594 4644 5000
SearchBox mount 920 916 5000
Shimmer mount 1862 1891 5000
Slider mount 1340 1317 5000
SpinButton mount 2912 2842 5000
Spinner mount 400 389 5000
SplitButton mount 1797 1813 5000
Stack mount 408 406 5000
StackWithIntrinsicChildren mount 874 892 5000
StackWithTextChildren mount 2686 2665 5000
SwatchColorPicker mount 6109 6097 5000
TagPicker mount 1438 1461 5000
Text mount 376 380 5000
TextField mount 955 936 5000
ThemeProvider mount 822 832 5000
ThemeProvider virtual-rerender 592 598 5000
ThemeProvider virtual-rerender-with-unmount 1279 1262 5000
Toggle mount 611 612 5000
buttonNative mount 198 193 5000

@Hotell Hotell marked this pull request as ready for review July 18, 2023 07:35
@Hotell Hotell requested a review from a team as a code owner July 18, 2023 07:35
@Hotell Hotell enabled auto-merge (squash) July 18, 2023 07:35
@Hotell Hotell merged commit 4ba8f4f into microsoft:master Jul 18, 2023
@Hotell Hotell deleted the hotell/ci/revert-bundlesize-pipeline branch July 18, 2023 11:47
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jul 18, 2023
* master:
  ci(bundlesize): opt out using --since as lightrail doesnt support this behaviour and needs everything build on every PR (microsoft#28545)
  feat: automatically add v9 package stories to public docsite and correctly create codesandbox demo source code (microsoft#28528)
  applying package updates
  Keytips: Align keytipData with visible instance for dupes (microsoft#28522)
  applying package updates
  V8 Fluent2 Theme: Spinner sizes (microsoft#28512)
  feat: add extra-tiny size value to size prop (microsoft#28249)
  feat(react-infobutton): Remove InfoIcon from react-infobutton (microsoft#28534)
  Made Breadcrumb package public (microsoft#28549)
  X bars showing incorrect data when the values are large- bug 8380 (microsoft#28510)
  feat(tools): implement `prepare-initial-release` generator (microsoft#28505)
  feat: release react-breadcrumb to preview (microsoft#28402)
  applying package updates
  fix: v8 SplitButton and split MenuItem have two touch targets when checkable (microsoft#28523)
  fix(react-infobutton): Apply aria-owns only when the popover is open and cleanup infobutton stories (microsoft#28463)
  fix: Pivot overflow role uses tab (microsoft#28409)
  Migrate react-search to preview (microsoft#28531)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Build System CI Fluent UI react (v8) Issues about @fluentui/react (v8) Fluent UI react-northstar (v0) Work related to Fluent UI V0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants