Blazity/next-enterprise

Suggestion: merge eslint and prettier

damianobarbati opened this issue · 2 comments

I suggest to merge the prettier configuration into eslint configuration.
This introduces several advantages:

  • only 1 tool needed
  • IDEs will rely on eslint only
  • lint --fix will both address linting and prettier inconsistencies
  • less commands to maintain, explain and to run

Here an example: https://github.com/damianobarbati/eslint-config-xs/blob/main/src/index.js

Hi, thanks for the issue!

That sounds like a good idea but I would like to keep the config in .prettierrc since it is kindof a standard and people expect the config to be there and not package.json, eslintrc.js. I think we can resolve that with import/require inside eslint config?

Are you open to creating a pull request for that?

Ok! As soon a I have time I'll submit a PR for this.