Skip to content
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

ci(.github): add issues write permisions to triage-bot worflow #24963

Merged
merged 1 commit into from
Sep 27, 2022

Conversation

Hotell
Copy link
Contributor

@Hotell Hotell commented Sep 27, 2022

Current Behavior

Triage bot throws 403 when running github-scripts

https://github.com/microsoft/fluentui/actions/runs/3134318373/jobs/5088716330#step:3:81
image

New Behavior

adding explicit permision for issues write

Related Issue(s)

follows #24911

@Hotell Hotell marked this pull request as ready for review September 27, 2022 11:08
@Hotell Hotell requested a review from a team as a code owner September 27, 2022 11:08
@Hotell Hotell added the CI label Sep 27, 2022
@Hotell Hotell enabled auto-merge (squash) September 27, 2022 11:09
@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 1afc810:

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

@size-auditor
Copy link

size-auditor bot commented Sep 27, 2022

Asset size changes

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

Baseline commit: af37ad65b2943761d035c10d14c87912b71f73ce (build)

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1287 1293 5000
Button mount 942 936 5000
FluentProvider mount 1501 1488 5000
FluentProviderWithTheme mount 578 581 10
FluentProviderWithTheme virtual-rerender 543 544 10
FluentProviderWithTheme virtual-rerender-with-unmount 590 589 10
MakeStyles mount 1967 1968 50000
SpinButton mount 2381 2350 5000

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-northstar)

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
TextMinimalPerf.default 311 291 1.07:1
FormMinimalPerf.default 340 324 1.05:1
ImageMinimalPerf.default 346 329 1.05:1
AttachmentMinimalPerf.default 131 126 1.04:1
ListNestedPerf.default 479 459 1.04:1
PortalMinimalPerf.default 141 136 1.04:1
RefMinimalPerf.default 193 186 1.04:1
DividerMinimalPerf.default 319 310 1.03:1
ListWith60ListItems.default 488 476 1.03:1
LoaderMinimalPerf.default 521 508 1.03:1
ProviderMinimalPerf.default 323 314 1.03:1
SegmentMinimalPerf.default 306 298 1.03:1
ToolbarMinimalPerf.default 807 785 1.03:1
AnimationMinimalPerf.default 483 474 1.02:1
BoxMinimalPerf.default 304 297 1.02:1
ChatDuplicateMessagesPerf.default 217 212 1.02:1
GridMinimalPerf.default 292 285 1.02:1
HeaderMinimalPerf.default 321 315 1.02:1
ItemLayoutMinimalPerf.default 983 966 1.02:1
LabelMinimalPerf.default 339 333 1.02:1
RadioGroupMinimalPerf.default 391 385 1.02:1
TooltipMinimalPerf.default 1899 1866 1.02:1
VideoMinimalPerf.default 619 609 1.02:1
CarouselMinimalPerf.default 357 353 1.01:1
DatepickerMinimalPerf.default 4704 4647 1.01:1
DropdownMinimalPerf.default 2171 2155 1.01:1
ProviderMergeThemesPerf.default 998 989 1.01:1
ReactionMinimalPerf.default 330 326 1.01:1
SplitButtonMinimalPerf.default 3282 3263 1.01:1
StatusMinimalPerf.default 613 604 1.01:1
TableManyItemsPerf.default 1577 1563 1.01:1
TableMinimalPerf.default 358 356 1.01:1
TextAreaMinimalPerf.default 411 407 1.01:1
CustomToolbarPrototype.default 2159 2137 1.01:1
AttachmentSlotsPerf.default 885 887 1:1
ButtonSlotsPerf.default 429 429 1:1
CardMinimalPerf.default 464 462 1:1
ChatMinimalPerf.default 630 630 1:1
EmbedMinimalPerf.default 2639 2637 1:1
FlexMinimalPerf.default 254 253 1:1
ListMinimalPerf.default 455 457 1:1
MenuButtonMinimalPerf.default 1344 1347 1:1
RosterPerf.default 1688 1695 1:1
SkeletonMinimalPerf.default 302 303 1:1
TreeMinimalPerf.default 703 701 1:1
CheckboxMinimalPerf.default 1527 1535 0.99:1
DialogMinimalPerf.default 685 691 0.99:1
HeaderSlotsPerf.default 670 675 0.99:1
InputMinimalPerf.default 841 849 0.99:1
LayoutMinimalPerf.default 309 312 0.99:1
MenuMinimalPerf.default 723 731 0.99:1
PopupMinimalPerf.default 546 554 0.99:1
IconMinimalPerf.default 563 567 0.99:1
AccordionMinimalPerf.default 124 126 0.98:1
AvatarMinimalPerf.default 158 161 0.98:1
ButtonMinimalPerf.default 131 133 0.98:1
DropdownManyItemsPerf.default 529 539 0.98:1
ListCommonPerf.default 507 518 0.98:1
SliderMinimalPerf.default 1216 1239 0.98:1
ButtonOverridesMissPerf.default 1000 1028 0.97:1
TreeWith60ListItems.default 128 132 0.97:1
ChatWithPopoverPerf.default 292 306 0.95:1
AlertMinimalPerf.default 216 231 0.94:1

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
global-context
createContext
533 B
341 B
global-context
createContextSelector
554 B
348 B
priority-overflow
createOverflowManager
2.936 kB
1.212 kB
react-accordion
Accordion (including children components)
78.914 kB
24.06 kB
react-alert
Alert
83.822 kB
21.029 kB
react-avatar
Avatar
48.692 kB
13.8 kB
react-avatar
AvatarGroup
14.95 kB
5.989 kB
react-avatar
AvatarGroupItem
68.66 kB
19.138 kB
react-badge
Badge
22.6 kB
7.205 kB
react-badge
CounterBadge
23.503 kB
7.497 kB
react-badge
PresenceBadge
24.05 kB
7.067 kB
react-button
Button
36.119 kB
9.647 kB
react-button
CompoundButton
43.144 kB
10.86 kB
react-button
MenuButton
38.813 kB
10.551 kB
react-button
SplitButton
46.228 kB
11.933 kB
react-button
ToggleButton
51.888 kB
11.127 kB
react-card
Card - All
67.002 kB
19.261 kB
react-card
Card
62.684 kB
18.177 kB
react-card
CardFooter
8.561 kB
3.601 kB
react-card
CardHeader
9.604 kB
3.94 kB
react-card
CardPreview
8.662 kB
3.656 kB
react-combobox
Combobox (including child components)
74.636 kB
24.186 kB
react-combobox
Dropdown (including child components)
74.236 kB
24.086 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
188.681 kB
52.366 kB
react-components
react-components: FluentProvider & webLightTheme
33.394 kB
11.007 kB
react-dialog
Dialog (including children components)
82.755 kB
24.581 kB
react-divider
Divider
16.459 kB
5.902 kB
react-image
Image
10.78 kB
4.264 kB
react-input
Input
23.757 kB
7.704 kB
react-label
Label
9.338 kB
3.86 kB
react-link
Link
11.784 kB
4.867 kB
react-menu
Menu (including children components)
116.589 kB
35.777 kB
react-menu
Menu (including selectable components)
119.658 kB
36.296 kB
react-overflow
hooks only
10.685 kB
4.104 kB
react-popover
Popover
102.955 kB
31.548 kB
react-portal
Portal
10.576 kB
3.875 kB
react-portal-compat
PortalCompatProvider
5.851 kB
1.964 kB
react-positioning
usePositioning
19.7 kB
7.404 kB
react-provider
FluentProvider
15.755 kB
5.883 kB
react-radio
Radio
35.56 kB
11.929 kB
react-radio
RadioGroup
14.248 kB
5.7 kB
react-select
Select
20.846 kB
7.346 kB
react-slider
Slider
31.526 kB
10.046 kB
react-spinbutton
SpinButton
44.102 kB
12.425 kB
react-spinner
Spinner
19.977 kB
6.438 kB
react-switch
Switch
32.097 kB
10.27 kB
react-text
Text - Default
11.782 kB
4.605 kB
react-text
Text - Wrappers
15.092 kB
5.044 kB
react-textarea
Textarea
25.013 kB
8.133 kB
react-theme
Single theme token import
69 B
89 B
react-theme
Teams: all themes
29.65 kB
6.444 kB
react-theme
Teams: Light theme
17.486 kB
5.057 kB
react-tooltip
Tooltip
41.535 kB
14.639 kB
react-utilities
SSRProvider
180 B
159 B
🤖 This report was generated against af37ad65b2943761d035c10d14c87912b71f73ce

@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 1212 1201 5000
Breadcrumb mount 2877 2780 1000
Checkbox mount 2708 2649 5000
CheckboxBase mount 2363 2387 5000
ChoiceGroup mount 4229 4272 5000
ComboBox mount 1174 1193 1000
CommandBar mount 9319 9251 1000
ContextualMenu mount 10301 10419 1000
DefaultButton mount 1344 1358 5000
DetailsRow mount 3388 3391 5000
DetailsRowFast mount 3409 3407 5000
DetailsRowNoStyles mount 3229 3264 5000
Dialog mount 2972 2958 1000
DocumentCardTitle mount 577 578 1000
Dropdown mount 3155 3196 5000
FocusTrapZone mount 1972 1988 5000
FocusZone mount 1924 1958 5000
GroupedList mount 48473 53012 2
GroupedList virtual-rerender 23202 23374 2
GroupedList virtual-rerender-with-unmount 81502 82578 2
GroupedListV2 mount 568 566 2
GroupedListV2 virtual-rerender 538 533 2
GroupedListV2 virtual-rerender-with-unmount 568 576 2
IconButton mount 1777 1816 5000
Label mount 745 761 5000
Layer mount 4170 4134 5000
Link mount 852 846 5000
MenuButton mount 1611 1590 5000
MessageBar mount 2372 2329 5000
Nav mount 3095 3102 1000
OverflowSet mount 1416 1413 5000
Panel mount 2523 2474 1000
Persona mount 1258 1264 1000
Pivot mount 1520 1545 1000
PrimaryButton mount 1511 1517 5000
Rating mount 7014 7013 5000
SearchBox mount 1514 1507 5000
Shimmer mount 2923 2971 5000
Slider mount 2093 2107 5000
SpinButton mount 4322 4298 5000
Spinner mount 830 840 5000
SplitButton mount 2853 2849 5000
Stack mount 902 897 5000
StackWithIntrinsicChildren mount 2249 2267 5000
StackWithTextChildren mount 4952 4940 5000
SwatchColorPicker mount 9625 9640 5000
TagPicker mount 2316 2320 5000
TeachingBubble mount 78019 78429 5000
Text mount 827 834 5000
TextField mount 1569 1559 5000
ThemeProvider mount 1448 1445 5000
ThemeProvider virtual-rerender 1146 1148 5000
ThemeProvider virtual-rerender-with-unmount 2002 2006 5000
Toggle mount 1139 1150 5000
buttonNative mount 530 536 5000

@Hotell Hotell merged commit 770c5e1 into microsoft:master Sep 27, 2022
@Hotell Hotell deleted the hotell/ci/triage-bot-permissions branch September 27, 2022 11:58
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Sep 28, 2022
* master: (21 commits)
  chore: Migrate react-avatar to use new build (microsoft#24969)
  applying package updates
  chore(react-input, react-textarea): Deprecating filled with shadow appearance variants (microsoft#24900)
  fix: v8 Dropdown no longer sets incorrect and unnecessary aria-activedescendant (microsoft#24593)
  feat: v0 Tooltip migration from v9 (microsoft#24908)
  chore: bump devDeps to fix critical security vulnerability (microsoft#24891)
  Fixing Tree chart issues (microsoft#24752)
  init: new package react-avatar-context (microsoft#24968)
  ci(.github): add issues write permisions to triage-bot worflow (microsoft#24963)
  applying package updates
  fix(Toolbar): close previous submenu when opening another submenu (microsoft#24836)
  fix: update non-focus-trap Popover role to be group (microsoft#24897)
  feat: Avatar's aria label includes 'active' or 'inactive' when using the active prop (microsoft#24901)
  feat(scripts): implement triage-bot module (microsoft#24911)
  chore: bump @octokit/rest to v18 (microsoft#24919)
  stress test: add "build-fixture" command (microsoft#24928)
  BREAKING-CHANGE: new ChatMessageContent for style caching (microsoft#24691)
  bugfix: fix changefile to properly update version of react-components with a patch (microsoft#24949)
  feat(scripts): enable strict checking for additional sub-folders(packages) (microsoft#24526)
  chore: exports DialogContent as unstable (microsoft#24943)
  ...
NotWoods pushed a commit to NotWoods/fluentui that referenced this pull request Nov 18, 2022
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