drop unicorn/better-regex because of a conflict with other rules

zloirock · Jun 9, 2021 · b553358 · b553358
1 parent 1aea2f3
commit b553358
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -365,8 +365,6 @@ const base = {
   strict: ['error', 'global'],
 
   // unicorn
-  // enforce the use of regex shorthands to improve readability
-  'unicorn/better-regex': 'error',
   // enforce a specific parameter name in catch clauses
   'unicorn/catch-error-name': ['error', { name: 'error', ignore: [/^err/] }],
   // enforce passing a message value when throwing a built-in error

diff --git a/tests/tests/es.regexp.flags.js b/tests/tests/es.regexp.flags.js
@@ -1,4 +1,4 @@
-/* eslint-disable unicorn/better-regex, regexp/sort-flags, regexp/no-useless-flag -- required for testing */
+/* eslint-disable regexp/sort-flags, regexp/no-useless-flag -- required for testing */
 import { DESCRIPTORS } from '../helpers/constants';
 
 if (DESCRIPTORS) {

diff --git a/tests/tests/es.regexp.to-string.js b/tests/tests/es.regexp.to-string.js
@@ -1,4 +1,4 @@
-/* eslint-disable unicorn/better-regex, regexp/sort-flags, regexp/no-useless-flag -- required for testing */
+/* eslint-disable regexp/sort-flags, regexp/no-useless-flag -- required for testing */
 import { STRICT } from '../helpers/constants';
 
 QUnit.test('RegExp#toString', assert => {