-
Notifications
You must be signed in to change notification settings - Fork 0
Consider removing eslint-plugin-prettier #1
Description
According to Prettier docs, it is "generally not recommended" to use eslint-plugin-prettier, versus having a separate prettier formatting workflow outside of eslint.
The link says:
The downsides of those plugins are:
- You end up with a lot of red squiggly lines in your editor, which gets annoying. Prettier is supposed to make you forget about formatting – and not be in your face about it!
- They are slower than running Prettier directly.
- They’re yet one layer of indirection where things may break.
In my personal, I have experienced tooling like Visual Studio Code break when using eslint-plugin-prettier and auto-fixing lint issues. What I've seen is that extra lines of code will be deleted, probably because VS Code is formatting on save (by default) and then running eslint --fix, which may be formatting on the stale file. So, for most recent projects I have followed the practices outlined on the Prettier site and haven't had problems.
We may want to look into a different way of sharing prettier configuration, so that projects can adopt this package without worrying about tradeoffs.
(Writing this issue as result of a retro item for the USCSF IWalk-Web engagement.)