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

chore(project): update format:diff ignore pattern #4409

Merged

Conversation

joshblack
Copy link
Contributor

Updates our format:diff to include react. The components pattern was mistakenly stopping this from occurring. Also updates failures from this change.

Changelog

New

Changed

  • format:diff now uses a separate ignore pattern with packages/components instead of components
  • Updated files according to prettier formatting

Removed

@joshblack joshblack requested a review from a team as a code owner October 21, 2019 20:32
@ghost ghost requested review from abbeyhrt and vpicone October 21, 2019 20:32
@joshblack
Copy link
Contributor Author

FYI @abbeyhrt, I think you might have been running into this

@abbeyhrt
Copy link
Contributor

Oh nice, thanks for looking into this! Should I wait for the to be merged before merging the endgame guide update?

@joshblack
Copy link
Contributor Author

@abbeyhrt I'm up for whatever ends up getting merged in first 👍 No harm either way

@netlify
Copy link

netlify bot commented Oct 21, 2019

Deploy preview for the-carbon-components ready!

Built with commit 6c556a6

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

@netlify
Copy link

netlify bot commented Oct 21, 2019

Deploy preview for carbon-components-react ready!

Built with commit 6c556a6

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

@netlify
Copy link

netlify bot commented Oct 21, 2019

Deploy preview for carbon-elements ready!

Built with commit 6c556a6

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

@joshblack joshblack merged commit d0e73ad into carbon-design-system:master Oct 21, 2019
@joshblack joshblack deleted the chore/update-format-check branch October 21, 2019 21:35
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.

2 participants