chore: add ESLint setup - #2585
Conversation
MikeMcC399
left a comment
There was a problem hiding this comment.
One non-blocking suggestion.
| "format:javascript": "eslint . --fix", | ||
| "format:javascript:check": "eslint .", |
There was a problem hiding this comment.
| "format:javascript": "eslint . --fix", | |
| "format:javascript:check": "eslint .", | |
| "format:javascript": "eslint --fix", | |
| "format:javascript:check": "eslint", |
In ESLint 10
You can also omit the file arguments and ESLint will use "."
Optional, non-blocking suggestion.
|
BTW: Whilst I was looking for a replacement for I don't want to suggest this alternative for a PR or merge, since I think the current setup is good enough as it is. It is just a Proof of Concept development to share with you. I'm OK with adding your PR which just does a base ESLint JS linting without additional plugins like stylistic. Prettier and @stylistic/eslint-plugin disagree about formatting, so it's not practical to use both on the same files. |
Description
Generated the config and ignored the one use that was still flagged inline.
Motivation and Context
Testing Details
Example Output(if appropriate)
Types of changes
Checklist