Consider using lint-staged

[copleykj]

It looks like preCommit.sh is basically doing what the lint-staged package does. Should we consider that instead?

[copleykj]

ping @StorytellerCZ

[StorytellerCZ]

🤔 I'm not sure. If we have a simple bash script that I think is preferable instead of adding additional package. It is also readable in regard what is going on and adjustable. On the other hand it is an extra file to maintain which might not be as bad with an extra package.

[copleykj]

I'm perfectly comfortable with a bash script, however lint-staged is a cross platform solution which will also work for devs working on windows based environments. Yes it's an extra package, but it's also just a dev dependency and wouldn't affect the build at all.

[StorytellerCZ]

OK, sounds good. Feel free to push the changes.

[StorytellerCZ]

@copleykj The only current problem I see that we currently have StandardJS, which is a problem for lint-staged:
Cannot add lint-staged: only eslint, stylelint, prettier or custom rules are supported.

[copleykj]

What about configuring eslint with eslint-plugin-standard?

[StorytellerCZ]

Probably the way forward so that we can include Meteor specific rules as well.