@kevinoid/eslint-config/rules/possible-errors.js

ESLint shareable config for @kevinoid's style (based on airbnb).

// ESLint common configuration rules related to possible errors
// https://eslint.org/docs/rules/#possible-errors

"use strict";

module.exports = {
  "rules": {
    // disallow expressions where the operation doesn't affect the value
    // TODO [eslint@>=9]: Remove.  Included in default config.
    "no-constant-binary-expression": "error",

    // disallow constant expressions in conditions
    "no-constant-condition": "error",

    // allow function or variable declarations in nested blocks
    // This is permitted in ES6 and accepted in all pre-ES6 engines I use.
    // As with vars-on-top rule, prefer declarations in scope of use for
    // ease of editing and comprehensibility.
    "no-inner-declarations": "off",

    // disallow new operators with global non-constructor functions
    // TODO [eslint-config-airbnb-base@>=16]: Remove if enabled
    "no-new-native-nonconstructor": "error",

    // disallow unused private class members
    // TODO [eslint@>=9]: Remove.  Included in default config.
    "no-unused-private-class-members": "error"
  }
};