Skip to content

Removing star exports from @fluentui/react-combobox#22800

Merged
khmakoto merged 1 commit intomicrosoft:masterfrom
khmakoto:reactComboboxStarExports
May 4, 2022
Merged

Removing star exports from @fluentui/react-combobox#22800
khmakoto merged 1 commit intomicrosoft:masterfrom
khmakoto:reactComboboxStarExports

Conversation

@khmakoto
Copy link
Member

@khmakoto khmakoto commented May 3, 2022

Current Behavior

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

New Behavior

react-combobox 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 553d1e2:

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

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
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: c45f4a6c3dd09c77ce5f5f5e2926ce9494169e99 (build)

Copy link
Member

@GeoffCoxMSFT GeoffCoxMSFT left a comment

Choose a reason for hiding this comment

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

I manually checked using auto-complete that nothing was missing from the exported sets.

@khmakoto khmakoto merged commit 2711113 into microsoft:master May 4, 2022
@khmakoto khmakoto deleted the reactComboboxStarExports branch May 4, 2022 19:37
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
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