Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: unpin icons-react dep in react package #4845

Merged
merged 5 commits into from
Dec 10, 2019

Conversation

vpicone
Copy link
Contributor

@vpicone vpicone commented Dec 10, 2019

closes #4844

@vpicone vpicone requested a review from a team as a code owner December 10, 2019 02:10
@ghost ghost requested review from asudoh and joshblack December 10, 2019 02:10
@vpicone
Copy link
Contributor Author

vpicone commented Dec 10, 2019

@joshblack Do we have a plan on migrating to the new icons package?

@netlify
Copy link

netlify bot commented Dec 10, 2019

Deploy preview for the-carbon-components ready!

Built with commit 414d450

https://deploy-preview-4845--the-carbon-components.netlify.com

@netlify
Copy link

netlify bot commented Dec 10, 2019

Deploy preview for carbon-elements ready!

Built with commit 414d450

https://deploy-preview-4845--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Dec 10, 2019

Deploy preview for carbon-components-react failed.

Built with commit 414d450

https://app.netlify.com/sites/carbon-components-react/deploys/5df013e730fcf0000882d1e5

@joshblack joshblack merged commit 2fd1098 into carbon-design-system:master Dec 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

carbon components react depends on a frozen version of @carbon/icons-react resulting in duplicate packages
3 participants