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: add name
to eslint configs
#18289
Conversation
✅ Deploy Preview for docs-eslint canceled.
|
2fe883f
to
f7b41e7
Compare
Looks like there are merge conflicts in |
f7b41e7
to
43b12ef
Compare
name
to exported configsname
to eslint configs
15af7ad
to
7e93954
Compare
0261706
to
8cca1dd
Compare
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. Would like another review before merging.
There's a merge conflict in |
8cca1dd
to
22a5528
Compare
22a5528
to
08f55ab
Compare
@@ -1771,7 +1771,7 @@ describe("cli", () => { | |||
getFixturePath("globals-node.js") | |||
]; | |||
|
|||
await cli.execute(`--no-eslintrc --config ./packages/js/src/configs/eslint-recommended.js --no-ignore ${files.join(" ")}`, null, false); | |||
await cli.execute(`--no-eslintrc --config ./tests/fixtures/config-file/js/.eslintrc.js --no-ignore ${files.join(" ")}`, null, false); |
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.
name
is not allowed in eslintrc mode. however, js.configs.recommended
was used here, so I updated the test to avoid failing.
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, thanks!
Part of the changes in this PR have been reverted in #18368 because they were causing some unit tests to fail. The tests started to fail only after upgrading the @eslint/js dependency in eslint's package.json. |
Prerequisites checklist
What is the purpose of this pull request? (put an "X" next to an item)
[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ x] Other, please explain:
What changes did you make? (Give an overview)
added name to config, per https://eslint.org/docs/head/use/configure/configuration-files#configuration-naming-conventions
Is there anything you'd like reviewers to focus on?
suggestions for the naming?