Skip to content

Conversation

@mperrotti
Copy link
Contributor

We were exporting octicons' deprecated default export.

@mperrotti mperrotti requested review from a team and colebemis March 16, 2022 16:28
@mperrotti mperrotti added the skip changeset This change does not need a changelog label Mar 16, 2022
@changeset-bot
Copy link

changeset-bot bot commented Mar 16, 2022

⚠️ No Changeset found

Latest commit: 91fc719

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Mar 16, 2022

size-limit report 📦

Path Size
dist/browser.esm.js 63.41 KB (0%)
dist/browser.umd.js 63.76 KB (0%)

Copy link
Contributor

@colebemis colebemis left a comment

Choose a reason for hiding this comment

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

Thank you for fixing this!

@colebemis colebemis merged commit 0487ecb into main Mar 16, 2022
@colebemis colebemis deleted the mp/fix-code-rendering-example branch March 16, 2022 18:05
pksjce pushed a commit that referenced this pull request Mar 17, 2022
* removes deprecated Octicons export from docs scope

* undo the change of  order in exports

* adds comment about why we do not export octicons-react default export

* Update docs/src/@primer/gatsby-theme-doctocat/live-code-scope.js

* Update export

Co-authored-by: Cole Bemis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip changeset This change does not need a changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants