Skip to content

Removing star exports from @fluentui/react-toolbar#22811

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

Removing star exports from @fluentui/react-toolbar#22811
khmakoto merged 1 commit into
microsoft:masterfrom
khmakoto:reactToolbarStarExports

Conversation

@khmakoto

@khmakoto khmakoto commented May 3, 2022

Copy link
Copy Markdown
Member

Current Behavior

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

New Behavior

react-toolbar 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 d7acfb0:

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

@fabricteam

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against 015ec324b72ceb7ae50a5545e3fc3f7064d10045

@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: 015ec324b72ceb7ae50a5545e3fc3f7064d10045 (build)

@khmakoto khmakoto merged commit 8d8e569 into microsoft:master May 4, 2022
@khmakoto khmakoto deleted the reactToolbarStarExports branch May 4, 2022 18:59
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.

4 participants