During the Emeris Extension movement from Manifest v2 -> v3, we have removed the eslint vite plugin from the vite config because it DELETED the file contents on error instead of fixing them.
Obviously we want to bring back eslint.
This is immediately an opportunity to try to make our standards match the web-app repo as much as possible.
Bonus: Probably the other packages would also benefit from some eslinting - maybe we should find a good way how to share eslint rules
faboweb
commented
2 years ago
During the Emeris Extension movement from Manifest v2 -> v3, we have removed the eslint vite plugin from the vite config because it DELETED the file contents on error instead of fixing them.
Obviously we want to bring back eslint. This is immediately an opportunity to try to make our standards match the web-app repo as much as possible.
Bonus: Probably the other packages would also benefit from some eslinting - maybe we should find a good way how to share eslint rules