eslint-plugin-ember

An ESlint plugin that provides set of rules for Ember Applications based on commonly known good practices.

❗️ Requirements

ESLint is required to use this plugin.
Rules provided by this plugin should be runnable under latest stable, v4 LTS and v6 LTS Node.js versions.

🚀 Usage

1. Install plugin

  yarn add --dev eslint-plugin-ember

  npm install --save-dev eslint-plugin-ember

2. Modify your `.eslintrc.js`:

Use with predefined settings:

// .eslintrc.js
module.exports = {
  plugins: [
    'ember'
  ],
  extends: [
    'eslint:recommended',
    'plugin:ember/recommended' // or 'plugin:ember/base'
  ],
  rules: {
    // override rules' settings here
  }
}

Possible configurations:

plugin:ember/base - contains no rules settings, but the basic eslint configuration suitable for any ember project. You can use it to configure rules as you wish.
plugin:ember/recommended - extends base configuration with recommended rules' settings

Use plain plugin:

If you don't want to use predefined settings, you can use it as a plain plugin:

module.exports = {
  plugins: [
    'ember'
  ],
  extends: [
    'eslint:recommended'
  ],
  rules: {
    // add rules' settings here, eg.:
    'ember/local-modules': 2
  }
}

All rules from this plugin have to be prefixed with ember/

🍟 Rules

Rules are grouped by category to help you understand their purpose.

All rules below with a check mark ✅ are enabled by default while using plugin:ember/recommended config.

The --fix option on the command line automatically fixes problems reported by rules which have a wrench 🔧 below.

Best Practices

	Rule ID	Description
	alias-model-in-controller	Enforces aliasing model in controller
✅	closure-actions	Enforces usage of closure actions
	named-functions-in-promises	Enforces usage of named functions in promises
✅	new-module-imports	Use "New Module Imports" from Ember RFC #176
	no-empty-attrs	Prevents usage of empty attributes in ember data models
✅	no-function-prototype-extensions	Prevents usage of Ember's `function` prototype extensions
✅	no-global-jquery	Prevents usage of global jQuery object
	no-jquery	Disallow any usage of jQuery
	no-observers	Prevents usage of observers
✅🔧	no-old-shims	Prevents usage of old shims for modules
✅	no-on-calls-in-components	Prevents usage of `on` calls in components
🔧	use-ember-get-and-set	Enforces usage of Ember.get and Ember.set

Possible Errors

	Rule ID	Description
✅	jquery-ember-run	Prevents usage of jQuery without Ember Run Loop
✅	no-attrs-in-components	Disallow usage of this.attrs in components
✅	no-attrs-snapshot	Disallow use of attrs snapshot in `didReceiveAttrs` and `didUpdateAttrs`
✅	no-capital-letters-in-routes	Raise an error when there is a route with uppercased letters in router.js
✅	no-duplicate-dependent-keys	Disallow repeating dependent keys
✅	no-side-effects	Warns about unexpected side effects in computed properties
✅	require-super-in-init	Enforces super calls in init hooks
✅	routes-segments-snake-case	Enforces usage of snake_cased dynamic segments in routes

Ember Object

	Rule ID	Description
✅	avoid-leaking-state-in-ember-objects	Avoids state leakage

Stylistic Issues

	Rule ID	Description
🔧	order-in-components	Enforces proper order of properties in components
🔧	order-in-controllers	Enforces proper order of properties in controllers
🔧	order-in-models	Enforces proper order of properties in models
🔧	order-in-routes	Enforces proper order of properties in routes
✅	use-brace-expansion	Enforces usage of brace expansion

Deprecated

⚠️ We're going to remove deprecated rules in the next major release. Please migrate to successor/new rules.

Rule ID	Replaced by
avoid-leaking-state-in-components	avoid-leaking-state-in-ember-objects
local-modules	new-module-imports

For the simplified list of rules, go here.

🍻 Contribution guide

In order to add a new rule, you should:

Create issue on GH with description of proposed rule
Generate a new rule using the official yeoman generator
Run yarn start
Write test scenarios & implement logic
Describe the rule in the generated docs file
Make sure all tests are passing
Run yarn run update in order to update readme and recommended configuration
Create PR and link created issue in description

Please be aware that we're using yarn in this repository, so if you plan to add some dependencies - make sure you commit yarn.lock file too.

If you have any suggestions, ideas or problems feel free to add new issue, but first please make sure your question does not repeat previous ones.

⭐️ Contributors

🙌 Credits

DockYard team - for great inspiration with their styleguide
Rob Hilgefort - for making it possible to redeploy new plugin under existing eslint-plugin-ember package name

🔓 License

See the LICENSE file for license rights and limitations (MIT).

alexgaribay / eslint-plugin-ember Goto Github PK