Skip to content

Conversation

@Hotell
Copy link
Contributor

@Hotell Hotell commented Sep 26, 2023

Previous Behavior

gitops bot rule contains rules which create and add weird labels on PRs that don't affect particular componnent

image

New Behavior

  • bot rules are removed to mitigate described issue
  • added explicit ownership and readme which points to bot docs

Related Issue(s)

  • Fixes #

@Hotell Hotell changed the title Hotell/ci/gitops bot ci(github/policies): remove bot rules that created and added Component/Package: labels on PRs Sep 26, 2023
@fabricteam
Copy link
Collaborator

fabricteam commented Sep 26, 2023

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
206.783 kB
59.084 kB
206.783 kB
59.085 kB

1 B
react-menu
Menu (including children components)
138.755 kB
42.859 kB
138.755 kB
42.86 kB

1 B
react-menu
Menu (including selectable components)
141.441 kB
43.369 kB
141.441 kB
43.37 kB

1 B
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.162 kB
1.735 kB
react-accordion
Accordion (including children components)
91.684 kB
28.007 kB
react-alert
Alert
83.657 kB
22.915 kB
react-avatar
Avatar
49.185 kB
15.491 kB
react-avatar
AvatarGroup
18.716 kB
7.458 kB
react-avatar
AvatarGroupItem
63.841 kB
19.834 kB
react-badge
Badge
25.921 kB
8.354 kB
react-badge
CounterBadge
26.822 kB
8.657 kB
react-badge
PresenceBadge
24.327 kB
8.867 kB
react-button
Button
39.444 kB
10.738 kB
react-button
CompoundButton
46.799 kB
12.233 kB
react-button
MenuButton
44.218 kB
12.12 kB
react-button
SplitButton
52.232 kB
13.705 kB
react-button
ToggleButton
56.49 kB
12.641 kB
react-card
Card - All
91.484 kB
26.485 kB
react-card
Card
86.31 kB
24.946 kB
react-card
CardFooter
12.074 kB
5.034 kB
react-card
CardHeader
14.323 kB
5.8 kB
react-card
CardPreview
13.031 kB
5.404 kB
react-checkbox
Checkbox
35.959 kB
11.852 kB
react-combobox
Combobox (including child components)
90.121 kB
29.441 kB
react-combobox
Dropdown (including child components)
88.561 kB
29.081 kB
react-components
react-components: Button, FluentProvider & webLightTheme
69.227 kB
19.584 kB
react-components
react-components: FluentProvider & webLightTheme
40.793 kB
13.521 kB
react-datepicker-compat
DatePicker Compat
210.557 kB
58.665 kB
react-dialog
Dialog (including children components)
88.307 kB
26.334 kB
react-divider
Divider
19.836 kB
7.392 kB
react-field
Field
21.443 kB
8.194 kB
react-image
Image
14.752 kB
5.875 kB
react-infobutton
InfoButton
128.844 kB
40.576 kB
react-infobutton
InfoLabel
132.535 kB
41.752 kB
react-input
Input
25.97 kB
8.77 kB
react-jsx-runtime
Classic Pragma
1.049 kB
531 B
react-jsx-runtime
JSX Dev Runtime
2.874 kB
1.307 kB
react-jsx-runtime
JSX Runtime
3.46 kB
1.542 kB
react-label
Label
13.159 kB
5.416 kB
react-link
Link
15.983 kB
6.492 kB
react-overflow
hooks only
12.419 kB
4.687 kB
react-persona
Persona
56.076 kB
17.369 kB
react-popover
Popover
117.589 kB
37.101 kB
react-portal
Portal
12.255 kB
4.504 kB
react-portal-compat
PortalCompatProvider
6.48 kB
2.203 kB
react-positioning
usePositioning
25.137 kB
9.109 kB
react-progress
ProgressBar
16.448 kB
6.545 kB
react-provider
FluentProvider
21.338 kB
7.945 kB
react-radio
Radio
29.72 kB
9.802 kB
react-radio
RadioGroup
14.355 kB
5.913 kB
react-select
Select
27.692 kB
9.849 kB
react-slider
Slider
36.87 kB
12.141 kB
react-spinbutton
SpinButton
35.857 kB
11.426 kB
react-spinner
Spinner
22.412 kB
8.118 kB
react-switch
Switch
32.179 kB
10.437 kB
react-table
DataGrid
155.92 kB
43.463 kB
react-table
Table (Primitives only)
42.715 kB
13.341 kB
react-table
Table as DataGrid
129.19 kB
34.771 kB
react-table
Table (Selection only)
74.557 kB
20.071 kB
react-table
Table (Sort only)
73.164 kB
19.67 kB
react-tags-preview
InteractionTag
14.275 kB
5.717 kB
react-tags-preview
Tag
28.998 kB
9.061 kB
react-tags-preview
TagGroup
72.507 kB
21.605 kB
react-text
Text - Default
15.766 kB
6.23 kB
react-text
Text - Wrappers
18.944 kB
6.551 kB
react-textarea
Textarea
30.03 kB
10.129 kB
react-theme
Single theme token import
69 B
89 B
react-theme
Teams: all themes
34.611 kB
7.295 kB
react-theme
Teams: Light theme
19.282 kB
5.486 kB
react-toast
Toast (including Toaster)
91.124 kB
27.293 kB
react-tooltip
Tooltip
50.859 kB
17.955 kB
react-utilities
SSRProvider
180 B
159 B
🤖 This report was generated against 3f9051784f483097b0c711495a61b0e634d880d6

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 26, 2023

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
InfoButton mount 10 12 5000 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 634 636 5000
Button mount 329 299 5000
Field mount 1118 1135 5000
FluentProvider mount 687 725 5000
FluentProviderWithTheme mount 87 89 10
FluentProviderWithTheme virtual-rerender 71 79 10
FluentProviderWithTheme virtual-rerender-with-unmount 77 77 10
InfoButton mount 10 12 5000 Possible regression
MakeStyles mount 862 883 50000
Persona mount 1758 1693 5000
SpinButton mount 1327 1410 5000

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 26, 2023

Perf Analysis (@fluentui/react-northstar)

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
AvatarMinimalPerf.default 111 98 1.13:1
ImageMinimalPerf.default 229 209 1.1:1
SkeletonMinimalPerf.default 205 188 1.09:1
VideoMinimalPerf.default 431 403 1.07:1
ProviderMinimalPerf.default 206 195 1.06:1
AccordionMinimalPerf.default 90 86 1.05:1
HeaderMinimalPerf.default 210 200 1.05:1
PortalMinimalPerf.default 86 82 1.05:1
BoxMinimalPerf.default 197 189 1.04:1
ButtonMinimalPerf.default 84 81 1.04:1
ListWith60ListItems.default 384 368 1.04:1
RadioGroupMinimalPerf.default 265 255 1.04:1
SegmentMinimalPerf.default 192 185 1.04:1
TableMinimalPerf.default 248 239 1.04:1
TreeWith60ListItems.default 85 82 1.04:1
MenuButtonMinimalPerf.default 963 939 1.03:1
CarouselMinimalPerf.default 264 258 1.02:1
DatepickerMinimalPerf.default 3562 3483 1.02:1
DropdownManyItemsPerf.default 390 382 1.02:1
LayoutMinimalPerf.default 201 197 1.02:1
ListNestedPerf.default 319 313 1.02:1
RosterPerf.default 1557 1524 1.02:1
ToolbarMinimalPerf.default 538 530 1.02:1
TooltipMinimalPerf.default 1270 1241 1.02:1
AnimationMinimalPerf.default 298 295 1.01:1
ButtonOverridesMissPerf.default 647 642 1.01:1
CardMinimalPerf.default 297 295 1.01:1
CheckboxMinimalPerf.default 1139 1127 1.01:1
DropdownMinimalPerf.default 1405 1392 1.01:1
ItemLayoutMinimalPerf.default 696 688 1.01:1
SplitButtonMinimalPerf.default 2260 2244 1.01:1
TreeMinimalPerf.default 468 463 1.01:1
ButtonSlotsPerf.default 311 312 1:1
ChatWithPopoverPerf.default 193 193 1:1
FormMinimalPerf.default 222 221 1:1
LabelMinimalPerf.default 221 222 1:1
ProviderMergeThemesPerf.default 643 646 1:1
ReactionMinimalPerf.default 206 206 1:1
CustomToolbarPrototype.default 1461 1454 1:1
ChatMinimalPerf.default 426 429 0.99:1
DividerMinimalPerf.default 199 202 0.99:1
EmbedMinimalPerf.default 1831 1844 0.99:1
ListMinimalPerf.default 296 298 0.99:1
PopupMinimalPerf.default 341 346 0.99:1
RefMinimalPerf.default 108 109 0.99:1
SliderMinimalPerf.default 736 741 0.99:1
IconMinimalPerf.default 382 385 0.99:1
TableManyItemsPerf.default 1088 1101 0.99:1
AttachmentSlotsPerf.default 646 658 0.98:1
GridMinimalPerf.default 180 183 0.98:1
HeaderSlotsPerf.default 450 460 0.98:1
TextMinimalPerf.default 189 193 0.98:1
AlertMinimalPerf.default 154 159 0.97:1
DialogMinimalPerf.default 428 442 0.97:1
InputMinimalPerf.default 527 541 0.97:1
ListCommonPerf.default 372 385 0.97:1
StatusMinimalPerf.default 380 390 0.97:1
TextAreaMinimalPerf.default 275 283 0.97:1
AttachmentMinimalPerf.default 88 92 0.96:1
LoaderMinimalPerf.default 173 180 0.96:1
MenuMinimalPerf.default 489 512 0.96:1
FlexMinimalPerf.default 148 157 0.94:1
ChatDuplicateMessagesPerf.default 139 149 0.93:1

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 26, 2023

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 77900e1:

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

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 26, 2023

Perf Analysis (@fluentui/react)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
BaseButton mount 610 626 5000
Breadcrumb mount 1673 1638 1000
Checkbox mount 1693 1686 5000
CheckboxBase mount 1456 1517 5000
ChoiceGroup mount 2953 2929 5000
ComboBox mount 646 657 1000
CommandBar mount 6074 6190 1000
ContextualMenu mount 11985 12000 1000
DefaultButton mount 724 759 5000
DetailsRow mount 2182 2142 5000
DetailsRowFast mount 2191 2199 5000
DetailsRowNoStyles mount 1989 2017 5000
Dialog mount 2591 2742 1000
DocumentCardTitle mount 229 219 1000
Dropdown mount 1981 1951 5000
FocusTrapZone mount 1123 1135 5000
FocusZone mount 1074 1031 5000
GroupedList mount 41211 41258 2
GroupedList virtual-rerender 19974 19768 2
GroupedList virtual-rerender-with-unmount 50408 50126 2
GroupedListV2 mount 216 232 2
GroupedListV2 virtual-rerender 207 202 2
GroupedListV2 virtual-rerender-with-unmount 222 228 2
IconButton mount 1081 1063 5000
Label mount 332 336 5000
Layer mount 2751 2717 5000
Link mount 383 390 5000
MenuButton mount 943 931 5000
MessageBar mount 21171 21308 5000
Nav mount 1904 1932 1000
OverflowSet mount 751 763 5000
Panel mount 1792 1706 1000
Persona mount 723 733 1000
Pivot mount 862 864 1000
PrimaryButton mount 852 827 5000
Rating mount 4517 4611 5000
SearchBox mount 914 897 5000
Shimmer mount 1893 1848 5000
Slider mount 1313 1299 5000
SpinButton mount 2860 2913 5000
Spinner mount 378 389 5000
SplitButton mount 1792 1828 5000
Stack mount 396 407 5000
StackWithIntrinsicChildren mount 846 849 5000
StackWithTextChildren mount 2585 2535 5000
SwatchColorPicker mount 6076 6127 5000
TagPicker mount 1434 1448 5000
Text mount 358 369 5000
TextField mount 937 910 5000
ThemeProvider mount 835 827 5000
ThemeProvider virtual-rerender 572 575 5000
ThemeProvider virtual-rerender-with-unmount 1272 1259 5000
Toggle mount 601 608 5000
buttonNative mount 186 187 5000

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 26, 2023

🕵 fluentuiv8 No visual regressions between this PR and main

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 26, 2023

🕵 FluentUIV0 No visual regressions between this PR and main

@size-auditor
Copy link

size-auditor bot commented Sep 26, 2023

Asset size changes

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

Baseline commit: 5a4c02bbedb845d1efbd593f793a6a32a1c53b4c (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 26, 2023

🕵 fluentuiv9 No visual regressions between this PR and main

@Hotell Hotell changed the title ci(github/policies): remove bot rules that created and added Component/Package: labels on PRs ci(github/policies): remove bot rules that created and added Component/Package $1: labels on PRs Sep 26, 2023
@Hotell Hotell marked this pull request as ready for review September 27, 2023 08:45
@Hotell Hotell added the CI label Sep 27, 2023
@Hotell Hotell enabled auto-merge (squash) September 27, 2023 08:49
@Hotell Hotell merged commit da9a088 into microsoft:master Oct 2, 2023
@Hotell Hotell deleted the hotell/ci/gitops-bot branch October 2, 2023 15:09
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Oct 4, 2023
* master: (146 commits)
  remove a11y test ux from theme designer (microsoft#29379)
  feat: preview release (microsoft#29377)
  Fixed divider in the tooltip (microsoft#29357)
  fix (microsoft#29376)
  docs: Initial documentation pass (microsoft#29372)
  feat: Integrate Announce context (microsoft#29362)
  chore(react-tags): stable release (microsoft#29355)
  chore: delegate focus outline to layout components (microsoft#29293)
  Vertical bar chart line bug (microsoft#29299)
  applying package updates
  Adding axis title examples for line, area, vertical bar and vertical stacked bar charts (microsoft#29254)
  A11y: Pagination: Add tooltip for all icon buttons  (microsoft#29155)
  A11y: Fix aria attribute for page number and add correct role (microsoft#29349)
  feat: Implement motion for MessageBar (microsoft#29339)
  feat: creates getIntrinsicElementProps to replace getNativeElementProps on slots creation (microsoft#29310)
  chore: replace makeStyles with makeResetStyles (microsoft#29338)
  applying package updates
  ci(github/policies): remove bot rules that created and added `Component/Package $1`: labels on PRs (microsoft#29301)
  fix: handle multiple classes in PortalCompatProvider (microsoft#29351)
  applying package updates
  ...
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Oct 4, 2023
* master: (49 commits)
  remove a11y test ux from theme designer (microsoft#29379)
  feat: preview release (microsoft#29377)
  Fixed divider in the tooltip (microsoft#29357)
  fix (microsoft#29376)
  docs: Initial documentation pass (microsoft#29372)
  feat: Integrate Announce context (microsoft#29362)
  chore(react-tags): stable release (microsoft#29355)
  chore: delegate focus outline to layout components (microsoft#29293)
  Vertical bar chart line bug (microsoft#29299)
  applying package updates
  Adding axis title examples for line, area, vertical bar and vertical stacked bar charts (microsoft#29254)
  A11y: Pagination: Add tooltip for all icon buttons  (microsoft#29155)
  A11y: Fix aria attribute for page number and add correct role (microsoft#29349)
  feat: Implement motion for MessageBar (microsoft#29339)
  feat: creates getIntrinsicElementProps to replace getNativeElementProps on slots creation (microsoft#29310)
  chore: replace makeStyles with makeResetStyles (microsoft#29338)
  applying package updates
  ci(github/policies): remove bot rules that created and added `Component/Package $1`: labels on PRs (microsoft#29301)
  fix: handle multiple classes in PortalCompatProvider (microsoft#29351)
  applying package updates
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants