Eslint rules discussion

[acabreragnz]

"rules": {

    // Our base eslint with airbnb
    "comma-dangle": [
      "error",
      "never"
    ],

    // Necessary to follow a clean code approach
    "no-use-before-define": [
      "error",
      {
        "functions": false,
        "classes": false
      }
    ],

   // Code can have empty space to separe block of things
    "no-trailing-spaces": [
      "error",
      {
        "skipBlankLines": true
      }
    ],

    // In express applications is very comon to modify the response
   // In general I think that is not a bad practice to update a property
    "no-param-reassign": [
      "error",
      {
        "props": false
      }
    ],

    // 80 is the default, Intellij allows 121
    "max-len": [
      "error",
      121
    ],

    // Don't allow ternary operator, why?
    "no-unused-expressions": [
      "error",
      {
        "allowTernary": true
      }
    ],

    // In express, error handlers are a special kind of middlewares with 4 arguments
   // In the last error handler is unnecesary to call the last argument (next), but it must appear
    "no-unused-vars": [
      "error",
      {
        "args": "none"
      }
    ],

    // I prefer {color: 'blue'} than { color: blue }
    "object-curly-spacing": [
      "error",
      "never"
    ],

    // I prefer function() than function ()
    "space-before-function-paren": [
      "error",
      "never"
    ]
}

In test directory we should define another .eslintrc.json with this rule

  "rules": {
    "no-unused-expressions": 0
  }

Specially if we are using chai.js, this kind of expect gives an error:
expect(null).to.be.null;

Let discuss about the rules I mentioned above, what do you think?

[rkmax]

I liked the changes you did, I'm using this one https://github.com/feross/eslint-config-standard but things like space-before-function-paren and object-curly-spacing are annoying for me, so 👍 x 💯

[rkmax]

@acabrera91 I would like to propose use the "standard" plus this rules, something like follows (sorry no comments), the following content is yours plus a couple of thing more, like babel and add the standard rules of javascript

{
  "parser": "babel-eslint",
  "extends": [
    "standard"
  ],
  "plugins": [
    "babel"
  ],
  "rules": {
    "key-spacing": 0,
    "max-len": [
      2,
      120,
      2
    ],
    "comma-dangle": [
      "error",
      "never"
    ],
    "no-use-before-define": [
      "error",
      {
        "functions": false,
        "classes": false
      }
    ],
    "no-param-reassign": [
      "error",
      {
        "props": false
      }
    ],
    "no-unused-expressions": [
      "error",
      {
        "allowTernary": true
      }
    ],
    "object-curly-spacing": [
      "error",
      "never"
    ],
    "space-before-function-paren": [
      "error",
      "never"
    ],
    "semi": ["error", "always"]
  }
}

For this one the dependencies will look like

...
    "eslint": "^3.15.0",
    "eslint-config-standard": "^6.2.1",
    "eslint-plugin-babel": "^4.0.1",
    "eslint-plugin-promise": "^3.4.2",
    "eslint-plugin-standard": "^2.0.1",
...

[jac1013]

Hello @acabrera91 I have doubts about this:

 // In express, error handlers are a special kind of middlewares with 4 arguments
 // In the last error handler is unnecesary to call the last argument (next), but it must appear
    "no-unused-vars": [
      "error",
      {
        "args": "none"
      }
    ],

imho what you are explaining in the comments is a problem with express, to me this rule make sense and helps to write better code.

[jac1013]

Hey guys, so what about joining what @rkmax improved in his comment and use what @acabrera91 created and start from that? Toni I'm still waiting on the response to the comment above, what do you think?