search

pluralsight / mob-timer

A mob programming application
Apache License 2.0
65 stars 26 forks source link

Add eslint with eslint:recommended #13

Closed dignite closed 6 years ago

dignite commented 6 years ago

Why?

Discussion in #9

Linting with eslint and having eslint rules regarding syntax and indentation is an alternative to prettier.

In order to speed up development And reduce time when onboarding And leaving more brain power to solving problems We want help with how to write and format our javascript

And while we don't agree with every rule The reward of consistency is worth the price

How?

This is a boilerplate to build upon with the rules that we consider useful.

Set no-unused-vars and no-undef to warn temporarily.

Next steps

The no-unused-vars and no-undef violations are potential bugs and should be fixed, and rules set to error.

More rules can be added, one option is ESlint Standard Style. https://github.com/standard/eslint-plugin-standard https://standardjs.com/

parkerholladay commented 6 years ago

In light of the conflicts with recent changes on master, I've opted to not wait for updates on this and instead use linting rules from a similar project. Closing...