Using standard code style

[expressjs]

Using standard code style

[dougwilson]

We're using Standard JS !

[LinusU]

I think it's a very good idea to move to standardjs, pretty much for the same two reasons that @dougwilson mentioned. I've started using them in all my projects since it helps to enforce a consistent coding style which absolutely helps when reading the code.

It also catches some programmer errors, such as accidentally leaking global variables, which is very good with many people contributing patches.

As already said, it's extremely easy to install and maintain, since there is no configuration and all upgrades are tested on the entire eco system.

Having standard as part of npm test helps eliminates unnecessary discussions about following the existing coding style on pull requests.

I'm all for implementing it, and would be happy to submit pull requests to do it 🙌

[hacksparrow]

+1 for standardjs.

[blakeembrey]

Sounds great. I've been using it for a year or so now, including on path-to-regexp (since I was deviating from the jshttp style accidentally at that point).

I'll need to update it to use the ESLint style configuration from our call today.

[dougwilson]

So here is an example of a commit I've made that sound help: expressjs/morgan@1533cdc

Of note are the following changes:

• Added a .eslintignore file, mainly to keep eslint from linting any generated coverage/ directory.
• Added a .eslintrc file, just specifying the standard style.
• Added npm run lint to the CI commands to run, specifically after the actual tests, such that it is easy to tell if a PR, even though it fails linting, passes the actual tests.
• Added devDependencies and a simple eslint command to package.json.
• Added a test/.eslintrc file for the mocha tests. Technically this can be a comment in the file, but ¯\_(ツ)_/¯

[jonathanong]

why not just run "lint": "eslint lib test" and forget about the .eslintignore file?

or eslint index.js test in this case

[dougwilson]

I tired that, but it mostly came down to editors. I had an editor that would let you run a report on your project and fix with eslint automatically, but as soon as I did the test coverage, it of course tried to do all this on those JS files from Istanbul output :(