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(sketch): update icon command with new metadata structure #5566

Conversation

joshblack
Copy link
Contributor

@joshblack joshblack commented Mar 6, 2020

This PR updates our icon command in our Sketch plugin. It seems like there were some updates between sketch versions that caused breaking changes (namely Rectangle and how it's used). In addition, this PR uses our new metadata structure which simplifies some of the logic for generating our icons page.

Changelog

New

Changed

  • Update sketch plugin with new metadata structure
  • Update sketch plugin wi th changes between sketch versions

Removed

Testing / Reviewing

  • Follow the steps in the Sketch package to link the plugin
  • Run the icons sync symbols command
  • Run the icons generate page command
  • Verify that both run and complete as expected

@joshblack joshblack requested a review from a team as a code owner March 6, 2020 23:10
@ghost ghost requested review from aledavila and tw15egan March 6, 2020 23:11
@joshblack
Copy link
Contributor Author

FYI @emyarod

@netlify
Copy link

netlify bot commented Mar 6, 2020

Deploy preview for carbon-components-react ready!

Built with commit f86dd20

https://deploy-preview-5566--carbon-components-react.netlify.com

@netlify
Copy link

netlify bot commented Mar 6, 2020

Deploy preview for carbon-elements ready!

Built with commit f86dd20

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

@netlify
Copy link

netlify bot commented Mar 6, 2020

Deploy preview for carbon-components-react ready!

Built with commit 3be39bb

https://deploy-preview-5566--carbon-components-react.netlify.com

@netlify
Copy link

netlify bot commented Mar 6, 2020

Deploy preview for carbon-elements ready!

Built with commit 3be39bb

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

Copy link
Member

@emyarod emyarod left a comment

Choose a reason for hiding this comment

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

after rebuilding the two icons commands appear to be working as expected

image

@joshblack joshblack merged commit c23a07d into carbon-design-system:master Mar 9, 2020
@joshblack joshblack deleted the fix/update-sketch-icon-command branch March 9, 2020 20:37
joshblack added a commit to joshblack/carbon that referenced this pull request Mar 10, 2020
…design-system#5566)

* chore: check-in work

* fix(sketch): update icon command with new metadata structure

Co-authored-by: Alessandra Davila <[email protected]>
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.

3 participants