Enable no-new eslint rule (#411)

yomidevs · Dec 21, 2023 · f041f80 · f041f80
1 parent 7dcf023
commit f041f80
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -62,6 +62,7 @@
         "no-const-assign": "error",
         "no-constant-condition": "off",
         "no-global-assign": "error",
+        "no-new": "error",
         "no-param-reassign": "off",
         "no-prototype-builtins": "error",
         "no-shadow": [

diff --git a/ext/js/pages/settings/profile-conditions-ui.js b/ext/js/pages/settings/profile-conditions-ui.js
@@ -389,6 +389,7 @@ export class ProfileConditionsUI extends EventDispatcher {
      */
     _validateRegExp(value) {
         try {
+            // eslint-disable-next-line no-new
             new RegExp(value, 'i');
             return true;
         } catch (e) {

diff --git a/ext/js/pages/settings/translation-text-replacements-controller.js b/ext/js/pages/settings/translation-text-replacements-controller.js
@@ -268,6 +268,7 @@ class TranslationTextReplacementsEntry {
         let okay = false;
         try {
             if (typeof value === 'string') {
+                // eslint-disable-next-line no-new
                 new RegExp(value, 'g');
                 okay = true;
             }