eslint-plugin-eggache
custom eslint rule for egg RTFM issues
Last updated 3 years ago by atian25 .
MIT · Repository · Bugs · Original npm
$ cnpm install eslint-plugin-eggache 
SYNC missed versions from official npm registry.

eslint-plugin-eggache

custom eslint rule for egg RTFM questions

NPM version build status Test coverage David deps Known Vulnerabilities NPM download

Usage

npm i eslint-plugin-eggache --save

Add eggache to the plugins section of your .eslintrc configuration file.

// ${app_root}/.eslintrc
{
  "extends": [
    "plugin:eggache/recommended"
  ]
}

By default it enable all the recommended rules, if you want to custom, just configure the rules section.

// ${app_root}/.eslintrc
{
  "extends": [
    "plugin:eggache/recommended"
  ],
  "rules": {
    'eggache/no-override-exports': [ 'error' ],
    'eggache/no-unexpected-plugin-keys': 'error',
  }
}

Rules

no-override-exports

A common mistake that newbie will make - override module.exports and exports.

/* eslint eggache/no-override-exports: [ 'error' ] */

// config/config.default.js
exports.view = {};

module.exports = appInfo => {
  const config = exports = {};
  config.keys = '123456';
  return config;
}

Options:

The first options is a boolean, default to false, means only check:

  • config/config.*.js
  • config/plugin.*.js

set it to true means to check all files.

/* eslint eggache/no-override-exports: [ 'error', true ] */

// due to options `true`, this will pass the lint
// ${app_root}/app.js
module.exports = exports = {};
exports.keys = '123456';

no-unexpected-plugin-keys

Sometimes, developer will confuse plugin.js and config.default.js.

plugin.js only allow [ 'enable', 'package', 'path', 'env' ] and it control whether to load a plugin.

The plugin's config should write to config/config.{env}.js.

/* eslint eggache/no-unexpected-plugin-keys: [ 'error' ] */

// config/plugin.js
module.exports = {
  test: {
    enable: true,
    package: 'egg-test',
    someConfig: 'should not place here',
  },
}

no-only-tests

A common mistake that developer will make - forget to remove .only in tests before committing.

/* eslint eggache/no-only-tests: [ 'error' ] */
describe.only('desc something', function() {
  it.only('assert somnething', function() {
    // do nothing
  });
})

Current Tags

  • 2.0.0                                ...           latest (3 years ago)

2 Versions

  • 2.0.0                                ...           3 years ago
  • 1.0.0                                ...           8 years ago
Maintainers (1)
Downloads
Today 0
This Week 0
This Month 0
Last Day 0
Last Week 0
Last Month 0
Dependencies (0)
None
Dev Dependencies (4)
Dependents (0)
None

Copyright 2013 - present © cnpmjs.org