-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Upgrade prettier and eslint compatibility libraries to latest #7478
Upgrade prettier and eslint compatibility libraries to latest #7478
Conversation
- upgrade prettier - upgrade eslint - upgrade eslint-plugin-prettier - upgrade eslint-config-prettier
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7478 +/- ##
=======================================
Coverage 55.50% 55.50%
=======================================
Files 671 671
Lines 26938 26938
Branches 2614 2614
=======================================
+ Hits 14951 14952 +1
+ Misses 11271 11269 -2
- Partials 716 717 +1
*This pull request uses carry forward flags. Click here to find out more. see 2 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
- The `prettier-eslint` vscode plugin sinc v6.0.0 no longer provides this package so we must install it as a devDependency so that autoformat works.
...and we have a new release of vscode-prettier-eslint that works! Huzzah! |
@@ -0,0 +1,14 @@ | |||
{ | |||
// See https://go.microsoft.com/fwlink/?LinkId=827846 to learn about workspace recommendations. |
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.
broken json
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.
huh. i think it still works but i'll remove it anyway
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.
It's jsonc actually. Just like tsconfig.json
.
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.
@JounQin hey, thanks!
@unlikelyzero i'm gonna revert this
This reverts commit 942f341.
Describe your changes:
In order to upgrade to prettier 3.x.x, we also need to upgrade the related eslint compatibility libraries (eslint, eslint-plugin-prettier, eslint-config-prettier). This PR does it all in one go, and runs lint:fix which only has one small change.
Also adds
.vscode/extensions.json
so that vscode can recommend extensions (such as prettier-eslint) for working in our codebaseAll Submissions:
Author Checklist
type:
label? Note: this is not necessarily the same as the original issue.Reviewer Checklist