Skip to content

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

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

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

Conversation

@khmakoto

@khmakoto khmakoto commented May 3, 2022

Copy link
Copy Markdown
Member

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

codesandbox-ci Bot commented May 3, 2022

Copy link
Copy Markdown

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
Copy Markdown
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

size-auditor Bot commented May 3, 2022

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: c45f4a6c3dd09c77ce5f5f5e2926ce9494169e99 (build)

@GeoffCoxMSFT GeoffCoxMSFT left a comment

Copy link
Copy Markdown
Member

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
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