-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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(linter): update eslint-config-prettier to v9 #19232
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 0303858. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 6 targets
Sent with 💌 from NxCloud. |
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.
Thank you for the PR.
I added version change to migration since it was missing.
Thanks for approving and keeping this PR up to date. Let me know if there's anything more that you want me to do |
Hopefully, we can get it in today. Thank you for the PR and sorry it's taking this long. |
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
@nx/eslint
uses eslint-config-prettier v8 as an optional peerDependency.Expected Behavior
@nx/eslint
uses eslint-config-prettier v9 as an optional peerDependency.Other instances of eslint-config-prettier have also been updated accordingly but the main focus for me was
@nx/eslint
Related Issue(s)
No issue has been created on this repo that is related to this as far as I could find.
There is a similar PR made to the
@vue/eslint-config-prettier
that does the same and this repo also needs to be updated once a new release for that is out.