Skip to content

Conversation

@Hotell
Copy link
Contributor

@Hotell Hotell commented Sep 23, 2022

Current Behavior

we use v 16. Danger uses v18

New Behavior

we use v18

NOTE:
Lot of things changed since v16 including types... those types moves are not very well documented, so I did the best I could. If we are no 100% confident we can drop this and use v16.

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 23, 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 06b7c59:

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

@size-auditor
Copy link

size-auditor bot commented Sep 23, 2022

Asset size changes

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

Baseline commit: 7ab1b2af48066e24961610b68762683c4698eb00 (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 23, 2022

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1281 1270 5000
Button mount 920 946 5000
FluentProvider mount 1481 1469 5000
FluentProviderWithTheme mount 578 584 10
FluentProviderWithTheme virtual-rerender 536 553 10
FluentProviderWithTheme virtual-rerender-with-unmount 580 564 10
MakeStyles mount 1960 1955 50000
SpinButton mount 2347 2310 5000

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 23, 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)
78.914 kB
24.06 kB
react-alert
Alert
83.511 kB
20.921 kB
react-avatar
Avatar
48.381 kB
13.696 kB
react-avatar
AvatarGroup
14.95 kB
5.989 kB
react-avatar
AvatarGroupItem
68.349 kB
19.039 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.689 kB
52.37 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.963 kB
31.553 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 7ab1b2af48066e24961610b68762683c4698eb00

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 23, 2022

Perf Analysis (@fluentui/react-northstar)

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
AccordionMinimalPerf.default 129 118 1.09:1
VideoMinimalPerf.default 627 588 1.07:1
AlertMinimalPerf.default 230 218 1.06:1
ButtonSlotsPerf.default 433 410 1.06:1
RefMinimalPerf.default 196 185 1.06:1
ChatDuplicateMessagesPerf.default 225 215 1.05:1
IconMinimalPerf.default 590 563 1.05:1
TableMinimalPerf.default 370 354 1.05:1
PortalMinimalPerf.default 143 137 1.04:1
AnimationMinimalPerf.default 479 463 1.03:1
AttachmentMinimalPerf.default 128 124 1.03:1
FormMinimalPerf.default 336 326 1.03:1
GridMinimalPerf.default 300 290 1.03:1
ProviderMinimalPerf.default 322 313 1.03:1
ButtonOverridesMissPerf.default 1033 1012 1.02:1
CarouselMinimalPerf.default 359 353 1.02:1
ChatMinimalPerf.default 647 635 1.02:1
LayoutMinimalPerf.default 318 311 1.02:1
DatepickerMinimalPerf.default 4743 4684 1.01:1
DropdownManyItemsPerf.default 544 541 1.01:1
FlexMinimalPerf.default 250 248 1.01:1
ImageMinimalPerf.default 343 340 1.01:1
InputMinimalPerf.default 869 861 1.01:1
ItemLayoutMinimalPerf.default 984 976 1.01:1
ListCommonPerf.default 519 515 1.01:1
ListNestedPerf.default 466 462 1.01:1
ListWith60ListItems.default 492 487 1.01:1
PopupMinimalPerf.default 561 554 1.01:1
SegmentMinimalPerf.default 298 296 1.01:1
TreeMinimalPerf.default 706 697 1.01:1
AttachmentSlotsPerf.default 880 882 1:1
AvatarMinimalPerf.default 162 162 1:1
BoxMinimalPerf.default 300 300 1:1
ButtonMinimalPerf.default 134 134 1:1
CheckboxMinimalPerf.default 1527 1529 1:1
DialogMinimalPerf.default 687 684 1:1
DividerMinimalPerf.default 316 317 1:1
DropdownMinimalPerf.default 2179 2173 1:1
EmbedMinimalPerf.default 2646 2637 1:1
HeaderMinimalPerf.default 313 312 1:1
HeaderSlotsPerf.default 682 682 1:1
SplitButtonMinimalPerf.default 3266 3265 1:1
StatusMinimalPerf.default 606 605 1:1
TextMinimalPerf.default 303 304 1:1
TextAreaMinimalPerf.default 417 418 1:1
CustomToolbarPrototype.default 2158 2154 1:1
TooltipMinimalPerf.default 1856 1855 1:1
ChatWithPopoverPerf.default 301 303 0.99:1
ListMinimalPerf.default 451 457 0.99:1
MenuMinimalPerf.default 724 734 0.99:1
MenuButtonMinimalPerf.default 1341 1361 0.99:1
RosterPerf.default 1716 1728 0.99:1
ProviderMergeThemesPerf.default 990 995 0.99:1
ReactionMinimalPerf.default 322 326 0.99:1
SkeletonMinimalPerf.default 298 300 0.99:1
TableManyItemsPerf.default 1555 1568 0.99:1
ToolbarMinimalPerf.default 792 799 0.99:1
TreeWith60ListItems.default 126 127 0.99:1
LabelMinimalPerf.default 335 343 0.98:1
RadioGroupMinimalPerf.default 383 389 0.98:1
SliderMinimalPerf.default 1228 1250 0.98:1
CardMinimalPerf.default 461 474 0.97:1
LoaderMinimalPerf.default 503 517 0.97:1

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 23, 2022

Perf Analysis (@fluentui/react)

Scenario Render type Master Ticks PR Ticks Iterations Status
ContextualMenu mount 6819 11491 1000 Possible regression
DefaultButton mount 1387 2486 5000 Possible regression
ThemeProvider virtual-rerender-with-unmount 2156 2133 5000 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
BaseButton mount 1180 1191 5000
Breadcrumb mount 2970 3009 1000
Checkbox mount 2538 2654 5000
CheckboxBase mount 2333 2295 5000
ChoiceGroup mount 4342 4377 5000
ComboBox mount 1252 1255 1000
CommandBar mount 9571 9578 1000
ContextualMenu mount 6819 11491 1000 Possible regression
DefaultButton mount 1387 2486 5000 Possible regression
DetailsRow mount 3666 5069 5000
DetailsRowFast mount 3671 3644 5000
DetailsRowNoStyles mount 3494 3478 5000
Dialog mount 3101 3107 1000
DocumentCardTitle mount 583 575 1000
Dropdown mount 3232 3244 5000
FocusTrapZone mount 2020 2013 5000
FocusZone mount 1972 1992 5000
GroupedList mount 53992 60976 2
GroupedList virtual-rerender 25654 25392 2
GroupedList virtual-rerender-with-unmount 93705 94857 2
GroupedListV2 mount 568 561 2
GroupedListV2 virtual-rerender 519 533 2
GroupedListV2 virtual-rerender-with-unmount 559 575 2
IconButton mount 1933 1918 5000
Label mount 735 734 5000
Layer mount 4270 4228 5000
Link mount 833 821 5000
MenuButton mount 1652 1693 5000
MessageBar mount 2298 2377 5000
Nav mount 3263 3273 1000
OverflowSet mount 1365 1352 5000
Panel mount 2555 2555 1000
Persona mount 1254 1290 1000
Pivot mount 1650 1674 1000
PrimaryButton mount 1519 1534 5000
Rating mount 7005 7073 5000
SearchBox mount 1506 1535 5000
Shimmer mount 2877 2896 5000
Slider mount 2089 2109 5000
SpinButton mount 4707 4592 5000
Spinner mount 800 784 5000
SplitButton mount 2993 3110 5000
Stack mount 851 860 5000
StackWithIntrinsicChildren mount 2312 2274 5000
StackWithTextChildren mount 4812 4741 5000
SwatchColorPicker mount 10476 10538 5000
TagPicker mount 2557 2634 5000
TeachingBubble mount 89870 89566 5000
Text mount 803 797 5000
TextField mount 1594 1595 5000
ThemeProvider mount 1512 1519 5000
ThemeProvider virtual-rerender 1079 1073 5000
ThemeProvider virtual-rerender-with-unmount 2156 2133 5000 Possible regression
Toggle mount 1120 1132 5000
buttonNative mount 548 554 5000

@Hotell Hotell marked this pull request as ready for review September 23, 2022 09:48
@Hotell Hotell requested a review from a team as a code owner September 23, 2022 09:48
@Hotell Hotell force-pushed the hotell/deps/bump-octokit branch from 94d83fc to ad1e817 Compare September 26, 2022 10:53
@Hotell Hotell enabled auto-merge (squash) September 26, 2022 11:32
@Hotell Hotell closed this Sep 26, 2022
auto-merge was automatically disabled September 26, 2022 16:22

Pull request was closed

@Hotell Hotell reopened this Sep 26, 2022
@Hotell Hotell merged commit b415287 into microsoft:master Sep 26, 2022
@Hotell Hotell deleted the hotell/deps/bump-octokit branch September 26, 2022 17:18
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
* chore: bump @octokit/rest to v18

* fixup! chore: bump @octokit/rest to v18
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.

4 participants