eslint-config-edx-es5
This is the edX ESLint shareable config for linting ES5 JavaScript. See eslint-config-edx
if you need to lint ES2015+ code.
Note that versions of this config prior to v2.0.0 are INCOMPATIBLE with eslint-config-edx.
edX maintains high-level documentation on the Open edX Developer's Guide on how we write new JavaScript features, including our use of RequireJS, Underscore and Backbone. This document seeks to be a lower-level guide on the semantics of how we write ES5 JavaScript, including nuts and bolts about spacing, syntax, and variable naming.
For the most part, edX follows the thoroughly documented Airbnb JavaScript Style Guide for ES5. Airbnb's JavaScript style is becoming the community standard, and their eslint-config-airbnb
is the most-downloaded ESLint config on NPM. Because the main Airbnb ESLint config assumes use of ES6, and edX's JavaScript is primarily written in ES5, this config is an extension of eslint-config-airbnb-base/legacy
, Airbnb's styleguide for ES5. Some of our rules also reference the Airbnb ES6 styleguide, the more modern and currently supported version.
In addition to the base Airbnb rules, edX adds or extends several of our own. They are described below.
dollar-sign
-
Setting:
["error", "ignoreProperties"]
-
Explanation: All variables that represent jQuery objects should be named starting with a
$
. Object properties may ignore this rule. -
Example:
// Correct patternvar $fooSpan = ;var ignoreProps = {};ignorePropsfooSpan = ;// Linter errorvar fooSpan = ;
func-names
- Setting:
"off"
- Explanation: Ignore this rule, it doesn't play nicely with RequireJS code.
indent
-
Setting:
["error", 4]
-
Explanation: edX is standardized on indenting all code with four spaces. The JavaScript community generally prefers two spaces; edX uses four because of our use of Python (see PEP8) and the desire to have consistency in our codebases.
-
Example:
// Correct patternvar {if numberOfSpaces !== 4throw 'Use four spaces for indentation.';};
max-len
- Setting:
["error", 120]
- Explanation: Use a maximum of 120 characters on a line. (The Airbnb default is 100, which we feel is too low.)
new-cap
-
Setting:
"off"
-
Explanation: Disable the rule stating functions can only start with a capital letter if they are constructors.
-
Example:
// Correct patternvar htmlFoo = HtmlUtils;
no-else-return
-
Setting:
"off"
-
Explanation: An
else
block after anif
which contains areturn
is allowed. Technically theelse
is redundant (and Airbnb disallows it), but this rule allows for more readable control flow. -
Example:
// Correct patternif fooString === 'foo'return 30;elsereturn 50;
no-shadow
-
Setting:
"error"
-
Explanation: Do not redefine variables in a child scope that are already declared in the upper scope.
-
Example:
// Linter errorvar foo = 'bar';if fizzvar foo = 'buzz';
object-curly-spacing
-
Setting:
["error", "never"]
-
Explanation: Do not insert extra spaces inside of curly brackets.
-
Example:
// Correct patternvar obj = foo: 42;// Linter errorvar obj2 = foo: 42 ;
one-var
-
Setting:
"off"
-
Explanation: Declare all variables at the top of your scope. You may use a single
var
statement (separating declarations by commas and newlines), or you may use multiplevar
statements. Just don't initialize multiple variables per line (see next rule). -
Example:
// All correct patterns{var foo bar bazfizz = 'buzz'foz = 49;if fizz// No more variable declarations below}{var foo bar baz;var fizz = 'buzz'foz = 49;if fizz// No more variable declarations below}{var foo;var bar;var baz;var fizz = 'buzz';var foz = 49;if fizz// No more variable declarations below}
one-var-declaration-per-line
-
Setting:
["error", "initializations"]
-
Explanation: Variables that are not initialized can be declared on the same line. Only one initialization is allowed per line.
-
Example:
// Correct patternvar foo barfizz = 'buzz'fozz = 'bizz';// Linter errorvar foo bar fizz = 'buzz' fozz = 'bizz';
space-before-function-paren
-
Setting:
["error", "never"]
-
Explanation: Do not add a space between a function and the opening parentheses containing its arguments, whether the function is anonymous or named.
-
Example:
// Correct patterns{}var {};// Linter errors{}var {};
strict
- Setting:
["error", "safe"]
- Explanation: If you're writing code that will run in the browser (i.e., AMD modules used by RequireJS), every top-level function declaration must have a
'use strict';
in it. Do not use'use strict';
anywhere else. If you're writing code that will run in Node (i.e., a CommonJS module, Karma config or Gulp task), use a single'use strict';
at global scope at the top of the module. - Example:
-
For browser code:
{'use strict';;}; -
For Node code:
/* eslint-env node */'use strict';var foo = ;;Instead of a
/* eslint-env node */
directive, you can also tell ESLint that a directory of.js
files should be treated as Node-specific code with a directory-specific.eslintrc.json
:project ├── .eslintrc.json ├── browser │ └── amd-module.js └── node ├── .eslintrc.json ├── commonjs-module-1.js └── commonjs-module-2.js
project/node/.eslintrc.json
:{ "env": "node" }
-