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

feat(pictograms): rename city asset names #5663

Merged

Conversation

joshblack
Copy link
Contributor

Closes #4117

Updates city names to contain one dash in their name.

Changelog

New

Changed

  • san--franscisco* -> san-frasncisco*
  • san--paolo* -> san-paolo*
  • washington--dc* -> washington-dc*
  • deprecated.yml has been updated with updates

Removed

@joshblack joshblack requested review from laurenmrice and a team as code owners March 19, 2020 16:40
@ghost ghost requested review from abbeyhrt and dakahn March 19, 2020 16:41
Copy link
Contributor

@dakahn dakahn left a comment

Choose a reason for hiding this comment

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

seems chill 🏄

@netlify
Copy link

netlify bot commented Mar 19, 2020

Deploy preview for carbon-elements ready!

Built with commit 8023f77

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

@joshblack
Copy link
Contributor Author

joshblack commented Mar 19, 2020

Note to reviewers, seems like these assets generate identical names to the ones that they're replacing. We might not need to do a deprecation pass if the exports are the same

@netlify
Copy link

netlify bot commented Mar 19, 2020

Deploy preview for carbon-components-react ready!

Built with commit 8023f77

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

Copy link
Member

@laurenmrice laurenmrice left a comment

Choose a reason for hiding this comment

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

There is a typo for second friendly name. Should be Francisco
Screen Shot 2020-03-19 at 4 25 05 PM

@joshblack
Copy link
Contributor Author

@laurenmrice fixed!

@joshblack joshblack merged commit 75abc2d into carbon-design-system:master Mar 19, 2020
@joshblack joshblack deleted the 4117-update-pictogram-names branch March 19, 2020 22:11
renmaddox pushed a commit to renmaddox/carbon that referenced this pull request Mar 20, 2020
* feat(pictograms): rename city asset names

* chore(pictograms): update asset names

* Update pictograms.yml
renmaddox added a commit to renmaddox/carbon that referenced this pull request Mar 20, 2020
renmaddox added a commit to renmaddox/carbon that referenced this pull request Mar 20, 2020
renmaddox pushed a commit to renmaddox/carbon that referenced this pull request Mar 20, 2020
* feat(pictograms): rename city asset names

* chore(pictograms): update asset names

* Update pictograms.yml
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.

chore(pictograms): correct delimiter for two word main names (.svg, categories.yml, and metadata.yml)
4 participants