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(deps): bump and fix prettier #13675
chore(deps): bump and fix prettier #13675
Conversation
✅ Deploy Preview for carbon-components-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, thanks for updating it!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, other than a merge conflict
5dfa566
to
deb8ab2
Compare
conflict solved, thanks |
* chore(deps): bump prettier to latest * fix: format files with prettier --------- Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
What
prettier
to latest.prettier
fromdevDependencies
in prettier-config-carbon.--cache
flag inlint-staged
.--cache
informat:diff
script.Why
--cache
option, which had a bug. However, this bug was fixed and released on V8prettier-config-carbon
is a shareable configuration that does not requireprettier
asdevDependency
. Instead, it should be added as apeerDependency
to indicate that this configuration requires versionv2.x
of theprettier
package.--cache
flag in my old pr.Changelog
Changed
prettier
to latest.Removed
prettier
fromdevDependencies
inprettier-config-carbon
Testing / Reviewing
CI should pass.