Unexpected results from curly: multi

What version of ESLint are you using?

3.3.1

What parser (default, Babel-ESLint, etc.) are you using?

Default

Please show your full configuration:

module.exports = {
    "env": {
        "node": true
    },
    "extends": "eslint:recommended",
    "rules": {
        "indent": [
            "error",
            2
        ],
        "linebreak-style": [
            "error",
            "unix"
        ],
        "quotes": [
            "error",
            "single"
        ],
        "semi": [
            "error",
            "always"
        ],
       "curly": [
          "error",
          "multi"
       ]
    }
};

What did you do? Please include the actual source code causing the issue.

/* global foo, cb */
if (foo) {
  try {
    foo();
  } catch (err) {
    cb(err);
  }
}

What did you expect to happen?

I would expect the above code to be valid.

What actually happened? Please include the actual, raw output from ESLint.

$ eslint index.js

/home/ryan/t-eslint/index.js
  2:1  error  Unnecessary { after 'if' condition  curly

✖ 1 problem (1 error, 0 warnings)

---

If this is the expected behavior, what is the correct way to format the code?

[RyanZim]

I don’t like:

if (foo)
    foo++;

Should be:

if (foo) foo++;

[RyanZim]

OK, I ended up wrapping my conditional inside the try block instead of putting the try statement in the conditional.

/* global foo, cb */

try {
  if (foo) foo();
} catch (err) {
  cb(err);
}

Closing for now.