sourcecred/.eslintrc.js

// @flow
module.exports = {
  extends: "react-app",
  rules: {
    "no-unused-vars": [
      "warn",
      {
        argsIgnorePattern: "^_$|^_unused_",
        varsIgnorePattern: "^_$|^_unused_",
        caughtErrorsIgnorePattern: "^_$|^_unused_",
      },
    ],
    "no-use-before-define": ["off"],
    "no-useless-constructor": ["off"],
  },
};
Add `@flow` to `.eslintrc.js` (#258) Summary: Even though it’s not really a source file, and it lives at the repository root, it might as well have typing to make sure that we don’t do anything really dumb. Test Plan: `yarn flow` still passes. wchargin-branch: flow-eslintrc 2018-05-10 12:27:46 -07:00			`// @flow`
Fix all lint errors, adding a lint CI step (#175) Test Plan: Run `yarn lint` and `yarn travis` and observe success. Add something that triggers a lint warning, like `const zzz = 3;`; re-run and observe failures. wchargin-branch: lint 2018-04-30 14:52:28 -07:00			`module.exports = {`
			`extends: "react-app",`
			`rules: {`
			`"no-unused-vars": [`
			`"warn",`
			`{`
			`argsIgnorePattern: "^_$\|^_unused_",`
			`varsIgnorePattern: "^_$\|^_unused_",`
			`caughtErrorsIgnorePattern: "^_$\|^_unused_",`
			`},`
			`],`
			`"no-use-before-define": ["off"],`
Disable the `no-useless-constructor` lint rule (#308) Summary: In fact, a constructor that only calls `super` is useful: it specifies the API for constructing an object of a given class without needing to traverse its prototype chain. (That constructors are inherited at all is arguably a design flaw.) Test Plan: None. wchargin-branch: yes-useful-constructor 2018-05-28 15:01:28 -07:00			`"no-useless-constructor": ["off"],`
Fix all lint errors, adding a lint CI step (#175) Test Plan: Run `yarn lint` and `yarn travis` and observe success. Add something that triggers a lint warning, like `const zzz = 3;`; re-run and observe failures. wchargin-branch: lint 2018-04-30 14:52:28 -07:00			`},`
			`};`