chore: add ESLint setup - #2585
Conversation
|
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. |
Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com> Co-authored-by: Nick Schonning <nschonni@gmail.com>
| "format:javascript": "eslint --fix", | ||
| "format:javascript:check": "eslint", |
There was a problem hiding this comment.
It would be more consistent to call these
lint:eslint
lint:eslint:fix
https://prettier.io/docs/comparison makes the distinction between formatters and linters
I am not hung up on the naming though. Anything that works, and is documented correctly is fine with me.
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