[x] This PR has a title that briefly describes the work done including the ticket number. If there is a ticket, make sure your PR title includes a conventional commit label. See existing PR titles for inspiration.
[ ] My work includes tests or is validated by existing tests.
Summary
Our current setup is running Prettier as a part of ESLint. These are two separate concerns that should be handled separately by different tools.
This PR removes eslint-plugin-prettier and eslint-config-prettier from our linting config as recommended here and here. I've also separated ESLint concerns from Prettier so that ESLint doesn't handle both. The recommended modern approach treats Prettier and ESLint as separate unrelated tools.
I've also removed the prettier script from the root-level package.json to the lint-staged config so that lint-staged runs Prettier after ESLint.
Requirements
Summary
Our current setup is running Prettier as a part of ESLint. These are two separate concerns that should be handled separately by different tools.
This PR removes eslint-plugin-prettier and eslint-config-prettier from our linting config as recommended here and here. I've also separated ESLint concerns from Prettier so that ESLint doesn't handle both. The recommended modern approach treats Prettier and ESLint as separate unrelated tools.
I've also removed the
prettier
script from the root-level package.json to thelint-staged
config so thatlint-staged
runs Prettier after ESLint.Screenshots
Related Issue
Other