No decisions to make. No .eslintrc, .jshintrc, or .jscsrc files to manage. It just
works.
This module saves you (and others!) time in two ways:
- No configuration. The easiest way to enforce consistent style in your module/project. Just drop it in.
- Catch style errors before they're submitted in PRs. Saves precious code review time by eliminating back-and-forth between maintainer and contributor.
Users of standard don't want to argue with you about your opinions about standard. That's the whole point of standard -- to avoid bikeshedding about style. Those of us who use standard find it helps hold our code to a high standard of quality and ensures that new contributors follow our module's style standards. The standard style laid out here is not affiliated with any official web standards groups, which is why this repo is called feross/standard and not ECMA/standard.
npm install standard- 2 spaces – for indentation
- Single quotes for strings – except to avoid escaping
- No unused variables – this one catches tons of bugs!
- No semicolons – It's fine. Really!
- Never start a line with
(or[- This is the only gotcha with omitting semicolons – automatically checked for you!
- Space after keywords
if (condition) { ... } - Space after function name
function name (arg) { ... } - Name the context variable
self–var self = this- Accidental
window.selfusage is dissallowed (happens whenvar self = thisis omitted)
- Accidental
- Always use
===instead of==– butobj == nullis allowed to checknull || undefined. - Always handle the node.js
errfunction parameter - Always prefix browser globals with
window– exceptdocumentandnavigatorare okay- Prevents accidental use of poorly-named browser globals like
open,length,event, andname.
- Prevents accidental use of poorly-named browser globals like
- And more goodness – give
standarda try today!
To get a better idea, take a look at
a sample file written
in JavaScript Standard Style, or check out some of the repositories that use standard.
Use this in one of your projects? Include one of these badges in your readme to let people know that your code is using the standard style.
[](https://github.com/feross/standard)[](https://github.com/feross/standard)The easiest way to use JavaScript Standard Style to check your code is to install it
globally as a Node command line program. To do so, simply run the following command in
your terminal (flag -g installs standard globally on your system, omit it if you want
to install in the current working directory):
npm install standard -gAfter you've done that you should be able to use the standard program. The simplest use
case would be checking the style of all JavaScript files in the current working directory:
$ standard
Error: Use JavaScript Standard Style
lib/torrent.js:950:11: Expected '===' and instead saw '=='.
First, install standard. Then, install the appropriate plugin for your editor:
- Vim - Install Syntastic and add
let g:syntastic_javascript_checkers = ['standard']to your.vimrc. - Sublime Text - Install Package Control, SublimeLinter, and SublimeLinter-contrib-standard.
- Atom - Install Linter and linter-js-standard.
- Add it to
package.json
{
"name": "my-cool-package",
"devDependencies": {
"standard": "^3.0.0"
},
"scripts": {
"test": "standard && node my-tests.js"
}
}- Check style automatically when you run
npm test
$ npm test
Error: Use JavaScript Standard Style
lib/torrent.js:950:11: Expected '===' and instead saw '=='.
- Never give style feedback on a pull request again!
The beauty of JavaScript Standard Style is that it's simple. No one wants to maintain multiple hundred-line style configuration files for every module/project they work on. Enough of this madness!
This module saves you time in two ways:
- No configuration. The easiest way to enforce consistent style in your module/project. Just drop it in.
- Catch style errors before they're submitted in PRs. Saves precious code review time by eliminating back-and-forth between maintainer and contributor.
The paths node_modules/, .git/, *.min.js, bundle.js, and coverage/ are automatically excluded
when looking for .js files to style check.
Sometimes you need to ignore additional folders or specific minfied files. To do that, add
a standard.ignore property to package.json:
"standard": {
"ignore": [
"**/out/**",
"**/lib/select2/**",
"**/lib/ckeditor/**"
]
}Yes! Just run standard --format filename.js. This uses
Max Ogden's automatic formatter
standard-format, which can automatically
fix most code issues.
While most issues can be fixed, some, like not handling errors, must be fixed manually.
In rare cases, you'll need to break a rule and hide the warning generated by standard.
JavaScript Standard Style uses eslint under-the-hood and you can
hide warnings as you normally would if you used eslint directly.
To get verbose output (so you can find the particular rule name to ignore), run:
$ standard --verbose
Error: Use JavaScript Standard Style
routes/error.js:20:36: 'file' was used before it was defined. (no-use-before-define)Disable all rules on a specific line:
file = 'I know what I am doing' // eslint-disable-lineOr, disable only the "no-use-before-define" rule:
file = 'I know what I am doing' // eslint-disable-line no-use-before-defineOr, disable the "no-use-before-define" rule for multiple lines:
/*eslint-disable no-use-before-define */
// offending code here...
// offending code here...
// offending code here...
/*eslint-enable no-use-before-define */No. Use eslint directly if you want that. Pro tip: Just use standard and move on.
There are actual real problems that you could spend your time solving! :P
standard prints to stderr. This means that tools that read from stdout won't be
able to read its output. The solution is to make standard print to stdout instead:
standard 2>&1 | grep variableWeb workers have a magic global variable called self. In regular JS files, standard
won't let you use self directly, as it wants to prevent accidental use of
window.self. But standard has no way of knowing when you are in a worker and
therefore does not know when to allow usage of self directly.
Until we figure out a better solution, we recommend adding this to the top of workers:
/* global self */
This lets standard (as well as humans reading your code) know that self is a global
in web worker code.
Funny you should ask!
#!/bin/sh
# Ensure all javascript files staged for commit pass standard code style
git diff --name-only --cached --relative | grep '\.js$' | xargs standard
exit $?MIT. Copyright (c) Feross Aboukhadijeh.