Add `ignore-catch` option to `no-empty`

What version of ESLint are you using? v2.7.0

Change proposal I would like to propose an ignore-catch option for the no-empty rule that would not warn for empty catch blocks. This would allow users the option to have have the same behavior as the corresponding JSCS rule disallowEmptyBlocks. The description of said rule is, “Disallows empty blocks (except for catch blocks).”

I’d be willing to make this change if it’s accepted.

[sindresorhus]

Duplicate of https://github.com/eslint/eslint/issues/2808 and #1841. I still think it’s a valid use-case though and it’s obvious now many users agree.

[nzakas]

That sounds fine. Compatibility with JSCS is important for people who are coming over, so I think @kaicataldo’s suggestion is good. The only change I’d make is that we’ve tried to move away from dash-cased option names, so I’d prefer something like:

no-empty: ["error", {
    allowEmptyCatch: true
}]

Where the default is false.