Try-catch statements should be a target of noUnusedParameters

[falsandtru]

Should use the optional catch clause instead.

TypeScript Version: master

Code

try {
}
catch (_) {
}

Expected behavior:

error

Actual behavior:

pass

Seems like a good idea but it would be a breaking change.

[mhegazy]

This is a breaking change. This check is a linter check in the first place, and we should not have added them. In the light of our recent discussions about errors vs warnings we should hold off on any of these changes.