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

chore: set @fluentui/react-storybook-addon dependency to a pinned version in various packages #26074

Merged

Conversation

TristanWatanabe
Copy link
Member

Changes:

  • sets @fluentui/react-storybook-addon dev dependency to a pinned version in v9, v9 docsite and v9 vr-tests packages. @fluentui/react-storybook-addon is still in prerelease and thus having it set as a caret can lead to non-deterministic resolutions.

Related Issue(s)

@size-auditor
Copy link

size-auditor bot commented Dec 22, 2022

Asset size changes

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

Baseline commit: 0dced6be8495f5341e09d8e0c5af7d20d9869b48 (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 1318 1313 5000
Button mount 938 835 5000
FluentProvider mount 1478 1505 5000
FluentProviderWithTheme mount 615 633 10
FluentProviderWithTheme virtual-rerender 597 587 10
FluentProviderWithTheme virtual-rerender-with-unmount 630 623 10
MakeStyles mount 1910 1886 50000
Persona mount 2838 2943 5000
SpinButton mount 2437 2474 5000

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
59.381 kB
16.465 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
187.067 kB
52.466 kB
react-components
react-components: FluentProvider & webLightTheme
33.843 kB
11.129 kB
react-portal-compat
PortalCompatProvider
5.857 kB
1.978 kB
🤖 This report was generated against 0dced6be8495f5341e09d8e0c5af7d20d9869b48

@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 ace0da5:

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

@fabricteam
Copy link
Collaborator

🕵 fluentuiv9 No visual regressions between this PR and main

@TristanWatanabe TristanWatanabe marked this pull request as ready for review December 22, 2022 03:50
@TristanWatanabe TristanWatanabe requested review from a team as code owners December 22, 2022 03:50
@ling1726
Copy link
Member

react-storybook-addon is not publicly released, so we would never really bump the version. Does it make sense to just mark it as stable?

@TristanWatanabe
Copy link
Member Author

react-storybook-addon is not publicly released, so we would never really bump the version. Does it make sense to just mark it as stable?

Thats a good point, what do you think @Hotell?

@TristanWatanabe TristanWatanabe merged commit 20221af into microsoft:master Dec 27, 2022
@TristanWatanabe TristanWatanabe deleted the set-storybook-addon-as-pinned branch December 27, 2022 17:13
q1b pushed a commit to q1b/fluentui that referenced this pull request Jan 24, 2023
Hotell pushed a commit to Hotell/fluentui that referenced this pull request Feb 9, 2023
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.

Set @fluentui/react-storybook-addon dependencies as pinned
5 participants