Skip to content

Removing star exports from @fluentui/react-context-selector#22801

Merged
khmakoto merged 2 commits intomicrosoft:masterfrom
khmakoto:reactContextSelectorStarExports
May 4, 2022
Merged

Removing star exports from @fluentui/react-context-selector#22801
khmakoto merged 2 commits intomicrosoft:masterfrom
khmakoto:reactContextSelectorStarExports

Conversation

@khmakoto
Copy link
Member

@khmakoto khmakoto commented May 3, 2022

Current Behavior

react-context-selector has export * from ... in src/index.ts.

New Behavior

react-context-selector has explicitly named exports in src/index.ts.

Related Issue(s)

#22099

@codesandbox-ci
Copy link

codesandbox-ci bot commented May 3, 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 9d615b9:

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

@fabricteam
Copy link
Collaborator

fabricteam commented May 3, 2022

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-accordion
Accordion (including children components)
73.754 kB
22.494 kB
react-combobox
Combobox
60.564 kB
20.751 kB
🤖 This report was generated against c45f4a6c3dd09c77ce5f5f5e2926ce9494169e99

@size-auditor
Copy link

size-auditor bot commented May 3, 2022

Asset size changes

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

Baseline commit: b8e02eabcec4e12fac1bee9ee5a7467710e7809e (build)

@fabricteam
Copy link
Collaborator

fabricteam commented May 3, 2022

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1152 1177 5000
Button mount 701 722 5000
FluentProvider mount 2244 2361 5000
FluentProviderWithTheme mount 351 346 10
FluentProviderWithTheme virtual-rerender 306 313 10
FluentProviderWithTheme virtual-rerender-with-unmount 399 379 10
MakeStyles mount 1836 1927 50000

@khmakoto khmakoto merged commit 982f68b into microsoft:master May 4, 2022
@khmakoto khmakoto deleted the reactContextSelectorStarExports branch May 4, 2022 18:55
@khmakoto khmakoto restored the reactContextSelectorStarExports branch May 4, 2022 18:55
@khmakoto khmakoto deleted the reactContextSelectorStarExports branch May 4, 2022 18:55
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
…t#22801)

* Removing star exports from @fluentui/react-context-selector.

* Adding change file.

Co-authored-by: KHMakoto <humberto_makoto@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants