Bug 1328929 - enable no-unsafe-finally eslint rule for devtools; r?jryans
MozReview-Commit-ID: J8v5YELIyid
--- a/devtools/.eslintrc.js
+++ b/devtools/.eslintrc.js
@@ -323,16 +323,18 @@ module.exports = {
// Disallow the use of Boolean literals in conditional expressions.
"no-unneeded-ternary": "error",
// Disallow unreachable statements after a return, throw, continue, or break
// statement.
"no-unreachable": "error",
// Disallow global and local variables that aren't used, but allow unused
// function arguments.
"no-unused-vars": ["error", {"vars": "all", "args": "none"}],
+ // Disallow flow control that escapes from "finally".
+ "no-unsafe-finally": "error",
// Allow using variables before they are defined.
"no-use-before-define": "off",
// We use var-only-at-top-level instead of no-var as we allow top level
// vars.
"no-var": "off",
// Allow using TODO/FIXME comments.
"no-warning-comments": "off",
// Disallow use of the with statement.