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

fix(web-components): enumerate side effects in package.json #28748

Conversation

chrisdholt
Copy link
Member

Previous Behavior

Some bundlers would tree shake out ambient imports which provide defined components.

New Behavior

Side effects are enumerated in the package.json.

@chrisdholt chrisdholt requested a review from a team as a code owner August 4, 2023 22:11
@fabricteam
Copy link
Collaborator

fabricteam commented Aug 4, 2023

🕵 fluentui-web-components-v3 No visual regressions between this PR and main

@fabricteam
Copy link
Collaborator

fabricteam commented Aug 4, 2023

📊 Bundle size report

🤖 This report was generated against 6642e27d0fa3037b40bf7659813416d115c5e987

@size-auditor
Copy link

size-auditor bot commented Aug 4, 2023

Asset size changes

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

Baseline commit: 6642e27d0fa3037b40bf7659813416d115c5e987 (build)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 4, 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 4289f1b:

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

@chrisdholt chrisdholt force-pushed the users/chhol/enumerate-side-effects-in-packagejson branch from f70a944 to 4289f1b Compare August 8, 2023 18:12
@chrisdholt chrisdholt merged commit a00e814 into microsoft:web-components-v3 Aug 8, 2023
21 checks passed
radium-v pushed a commit to radium-v/fluentui that referenced this pull request Apr 29, 2024
radium-v pushed a commit to radium-v/fluentui that referenced this pull request Apr 29, 2024
radium-v pushed a commit to radium-v/fluentui that referenced this pull request Apr 30, 2024
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.

None yet

3 participants