a7c3268b79
This disables the no-undefined rule. Most JavaScript projects in OpenStack, at this point, have adopted eslint-config-angular, which declares the 'definedundefined' rule requiring angular.isDefined() or angular.isUndefined() to handle this case. That makes this rule redundant, while also forcing non-production pieces of code (such as unit tests) to become unnecessarily verbose. https://github.com/Gillespie59/eslint-plugin-angular/blob/master/docs/definedundefined.md http://eslint.org/docs/rules/no-undefined Change-Id: I35ae5c8bc9e7ebb5c455d4acbe79636653448f65 |
||
---|---|---|
spec | ||
.editorconfig | ||
.eslintrc | ||
.gitignore | ||
.gitreview | ||
.npmrc | ||
LICENSE | ||
README.md | ||
index.js | ||
package.json |
README.md
eslint-config-openstack
OpenStack has a set of style guidelines for clarity. OpenStack is a very large code base, spanning dozens of git trees, with over a thousand developers contributing every 6 months. As such, common style helps developers understand code in reviews, move between projects smoothly, and overall make the code more maintainable.
Even though eslint permits overriding rules on a per-project basis, it should be the goal of every project to stay as close to the common guidelines as possible.
Installation
npm install --save-dev eslint eslint-config-openstack
- Add
extends: "openstack"
to your.eslintrc