Skip to content

Enable screener workflows#24178

Merged
ralucapelin merged 37 commits intomasterfrom
enable-screener-workflows
Aug 8, 2022
Merged

Enable screener workflows#24178
ralucapelin merged 37 commits intomasterfrom
enable-screener-workflows

Conversation

@ralucapelin
Copy link
Contributor

@ralucapelin ralucapelin commented Aug 2, 2022

Current Behavior

Screener runs are triggered via Azure DevOps pipelines.

New Behavior

Screener runs are triggered via GitHub Actions workflows added in PR #24106.

Related Issue(s)

Addresses #23903

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 2, 2022

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

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

@fabricteam
Copy link
Collaborator

fabricteam commented Aug 2, 2022

📊 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)
79.485 kB
24.082 kB
react-alert
Alert
82.763 kB
20.582 kB
react-avatar
Avatar
48.172 kB
13.615 kB
react-avatar
AvatarGroup
13.43 kB
5.382 kB
react-avatar
AvatarGroupItem
64.878 kB
18.284 kB
react-badge
Badge
22.494 kB
7.157 kB
react-badge
CounterBadge
23.397 kB
7.449 kB
react-badge
PresenceBadge
23.947 kB
7.022 kB
react-button
Button
36.396 kB
9.575 kB
react-button
CompoundButton
43.469 kB
10.812 kB
react-button
MenuButton
39.014 kB
10.456 kB
react-button
SplitButton
46.506 kB
11.827 kB
react-button
ToggleButton
51.91 kB
11.003 kB
react-card
Card - All
67.42 kB
19.249 kB
react-card
Card
63.102 kB
18.167 kB
react-card
CardFooter
8.461 kB
3.555 kB
react-card
CardHeader
9.504 kB
3.896 kB
react-card
CardPreview
8.562 kB
3.61 kB
react-combobox
Combobox (including child components)
75.081 kB
24.145 kB
react-combobox
Dropdown (including child components)
74.594 kB
24.134 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
192.719 kB
52.784 kB
react-components
react-components: FluentProvider & webLightTheme
32.688 kB
10.736 kB
react-dialog
Dialog (including children components)
85.064 kB
25.293 kB
react-divider
Divider
16.321 kB
5.837 kB
react-image
Image
10.68 kB
4.215 kB
react-input
Input
23.554 kB
7.644 kB
react-label
Label
9.238 kB
3.815 kB
react-link
Link
12.197 kB
4.912 kB
react-menu
Menu (including children components)
118.878 kB
36.004 kB
react-menu
Menu (including selectable components)
122.077 kB
36.497 kB
react-overflow
hooks only
10.839 kB
4.146 kB
react-popover
Popover
106.135 kB
32.236 kB
react-portal
Portal
10.49 kB
3.845 kB
react-positioning
usePositioning
23.854 kB
8.299 kB
react-provider
FluentProvider
15.565 kB
5.818 kB
react-radio
Radio
36.13 kB
11.947 kB
react-radio
RadioGroup
14.319 kB
5.711 kB
react-select
Select
20.746 kB
7.299 kB
react-slider
Slider
31.988 kB
10.019 kB
react-spinbutton
SpinButton
43.899 kB
12.362 kB
react-spinner
Spinner
19.932 kB
6.363 kB
react-switch
Switch
32.438 kB
10.218 kB
react-text
Text - Default
11.572 kB
4.537 kB
react-text
Text - Wrappers
14.882 kB
4.977 kB
react-textarea
Textarea
23.078 kB
7.707 kB
react-theme
Single theme token import
69 B
89 B
react-theme
Teams: all themes
28.995 kB
6.215 kB
react-theme
Teams: Light theme
16.973 kB
4.86 kB
react-tooltip
Tooltip
45.515 kB
15.545 kB
react-utilities
SSRProvider
189 B
161 B
🤖 This report was generated against 5d2617e9b90046a13a6e493c1e54c4b45101f509

@fabricteam
Copy link
Collaborator

fabricteam commented Aug 2, 2022

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1302 1262 5000
Button mount 974 968 5000
FluentProvider mount 1599 1599 5000
FluentProviderWithTheme mount 630 636 10
FluentProviderWithTheme virtual-rerender 592 598 10
FluentProviderWithTheme virtual-rerender-with-unmount 635 665 10
MakeStyles mount 2033 1941 50000
SpinButton mount 2538 2482 5000

@size-auditor
Copy link

size-auditor bot commented Aug 2, 2022

Asset size changes

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

Baseline commit: 5d2617e9b90046a13a6e493c1e54c4b45101f509 (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Aug 2, 2022

Perf Analysis (@fluentui/react-northstar)

⚠️ 1 potential perf regressions detected

Potential regressions comparing to master

Scenario Current PR Ticks Baseline Ticks Ratio Regression Analysis
ButtonMinimalPerf.default 136 114 1.19:1 analysis
Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
ListMinimalPerf.default 440 377 1.17:1
ReactionMinimalPerf.default 314 268 1.17:1
VideoMinimalPerf.default 539 475 1.13:1
FormMinimalPerf.default 327 294 1.11:1
TableManyItemsPerf.default 1568 1425 1.1:1
AnimationMinimalPerf.default 441 407 1.08:1
SegmentMinimalPerf.default 283 264 1.07:1
TreeWith60ListItems.default 132 124 1.06:1
ChatWithPopoverPerf.default 326 310 1.05:1
DropdownManyItemsPerf.default 559 533 1.05:1
HeaderMinimalPerf.default 304 290 1.05:1
ItemLayoutMinimalPerf.default 961 913 1.05:1
LoaderMinimalPerf.default 576 547 1.05:1
RefMinimalPerf.default 167 159 1.05:1
CustomToolbarPrototype.default 2316 2215 1.05:1
DatepickerMinimalPerf.default 4987 4801 1.04:1
HeaderSlotsPerf.default 628 603 1.04:1
SliderMinimalPerf.default 1434 1383 1.04:1
ToolbarMinimalPerf.default 732 703 1.04:1
AccordionMinimalPerf.default 128 124 1.03:1
AvatarMinimalPerf.default 151 147 1.03:1
DropdownMinimalPerf.default 2599 2528 1.03:1
InputMinimalPerf.default 1105 1068 1.03:1
ProviderMergeThemesPerf.default 1072 1040 1.03:1
ProviderMinimalPerf.default 345 336 1.03:1
SplitButtonMinimalPerf.default 3576 3464 1.03:1
ButtonSlotsPerf.default 452 443 1.02:1
ChatMinimalPerf.default 613 599 1.02:1
DividerMinimalPerf.default 299 292 1.02:1
LabelMinimalPerf.default 322 315 1.02:1
ListCommonPerf.default 523 515 1.02:1
ListNestedPerf.default 470 462 1.02:1
MenuButtonMinimalPerf.default 1408 1383 1.02:1
RadioGroupMinimalPerf.default 382 375 1.02:1
SkeletonMinimalPerf.default 284 278 1.02:1
AttachmentSlotsPerf.default 807 798 1.01:1
DialogMinimalPerf.default 643 639 1.01:1
CardMinimalPerf.default 441 443 1:1
ChatDuplicateMessagesPerf.default 246 245 1:1
FlexMinimalPerf.default 241 242 1:1
LayoutMinimalPerf.default 291 291 1:1
TableMinimalPerf.default 331 330 1:1
ImageMinimalPerf.default 300 303 0.99:1
MenuMinimalPerf.default 713 718 0.99:1
TextAreaMinimalPerf.default 392 397 0.99:1
TreeMinimalPerf.default 636 644 0.99:1
AttachmentMinimalPerf.default 125 128 0.98:1
CarouselMinimalPerf.default 382 388 0.98:1
EmbedMinimalPerf.default 3338 3404 0.98:1
RosterPerf.default 903 917 0.98:1
PopupMinimalPerf.default 519 527 0.98:1
IconMinimalPerf.default 503 517 0.97:1
ListWith60ListItems.default 474 496 0.96:1
TooltipMinimalPerf.default 843 881 0.96:1
BoxMinimalPerf.default 267 280 0.95:1
CheckboxMinimalPerf.default 2068 2185 0.95:1
StatusMinimalPerf.default 512 549 0.93:1
ButtonOverridesMissPerf.default 1116 1232 0.91:1
GridMinimalPerf.default 242 271 0.89:1
AlertMinimalPerf.default 198 225 0.88:1
TextMinimalPerf.default 250 284 0.88:1
PortalMinimalPerf.default 123 143 0.86:1

@fabricteam
Copy link
Collaborator

fabricteam commented Aug 2, 2022

Perf Analysis (@fluentui/react)

Scenario Render type Master Ticks PR Ticks Iterations Status
TeachingBubble mount 48480 82648 5000 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
BaseButton mount 809 704 5000
Breadcrumb mount 2200 2189 1000
Checkbox mount 2017 2135 5000
CheckboxBase mount 1973 1960 5000
ChoiceGroup mount 3830 3731 5000
ComboBox mount 812 802 1000
CommandBar mount 8972 9031 1000
ContextualMenu mount 9689 10253 1000
DefaultButton mount 923 934 5000
DetailsRow mount 3000 3090 5000
DetailsRowFast mount 3183 2990 5000
DetailsRowNoStyles mount 2936 2944 5000
Dialog mount 2229 2371 1000
DocumentCardTitle mount 153 135 1000
Dropdown mount 2658 2786 5000
FocusTrapZone mount 1569 1542 5000
FocusZone mount 1518 1587 5000
IconButton mount 1402 1462 5000
Label mount 313 263 5000
Layer mount 2620 2483 5000
Link mount 398 371 5000
MenuButton mount 1157 1223 5000
MessageBar mount 1763 1787 5000
Nav mount 2818 2588 1000
OverflowSet mount 916 920 5000
Panel mount 1861 1832 1000
Persona mount 823 844 1000
Pivot mount 1201 1262 1000
PrimaryButton mount 1051 1157 5000
Rating mount 6424 6570 5000
SearchBox mount 1073 1099 5000
Shimmer mount 2134 2089 5000
Slider mount 1663 1540 5000
SpinButton mount 4120 4227 5000
Spinner mount 377 342 5000
SplitButton mount 2654 2777 5000
Stack mount 404 449 5000
StackWithIntrinsicChildren mount 1949 1985 5000
StackWithTextChildren mount 4415 4468 5000
SwatchColorPicker mount 9690 9640 5000
TagPicker mount 2210 2466 5000
TeachingBubble mount 48480 82648 5000 Possible regression
Text mount 361 378 5000
TextField mount 1200 1186 5000
ThemeProvider mount 934 1058 5000
ThemeProvider virtual-rerender 583 563 5000
ThemeProvider virtual-rerender-with-unmount 1512 1508 5000
Toggle mount 690 693 5000
buttonNative mount 104 115 5000

Raluca Pelin added 5 commits August 2, 2022 14:24
author Raluca Pelin <t-rpelin@microsoft.com> 1659439645 +0300
committer Raluca Pelin <t-rpelin@microsoft.com> 1659439711 +0300

parent e25a471
author Raluca Pelin <t-rpelin@microsoft.com> 1659439645 +0300
committer Raluca Pelin <t-rpelin@microsoft.com> 1659439665 +0300

Solve rebase conflict

Remove screener jobs from pipeline

Additional documentation

Add checkout action option

Add change suggested by review

Restore pipeline from master branch

Make changes proposed in review

Update step name

Co-authored-by: ling1726 <lingfangao@hotmail.com>

Fix typo for environment variable value

Change workflow trigger event and remove ADO screener jobs

Comment out screener code check

Remove screener code check
@ralucapelin ralucapelin marked this pull request as ready for review August 4, 2022 08:20
@ralucapelin ralucapelin requested a review from a team as a code owner August 4, 2022 08:20
displayName: Cypress E2E tests

- template: .devops/templates/cleanup.yml

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍 the DAY has arrived

@ralucapelin ralucapelin requested a review from Hotell August 5, 2022 07:19
This was referenced Aug 5, 2022
Copy link
Contributor

@Hotell Hotell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, mind double-check one more time @ling1726 🙏?

@ralucapelin ralucapelin merged commit 596ed97 into master Aug 8, 2022
@ralucapelin ralucapelin deleted the enable-screener-workflows branch August 8, 2022 13:03
layershifter pushed a commit to layershifter/office-ui-fabric-react that referenced this pull request Aug 8, 2022
This reverts commit 596ed97.

Co-authored-by: ling1726 <lingfangao@hotmail.com>
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.

5 participants