Skip to content

Commit d015e9c

Browse files
authored
fix: re-enable no-control-regex (#3)
It wasn't immediately obvious to me why we'd have this off by default since oxlint enables it by default: https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-control-regex.html It does appear that there is code in expo/expo that disables the eslint rule explicitly: https://github.com/search?q=repo%3Aexpo%2Fexpo%20no-control-regex&type=code. I think this order of precedence is correct. On by default to match oxlint default, explicit per-case disable.
1 parent 030071d commit d015e9c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

shared/core.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ export default defineConfig({
3636
'no-const-assign': 'error',
3737
'no-constant-binary-expression': 'warn',
3838
'no-constant-condition': ['warn', { checkLoops: false }],
39-
'no-control-regex': 'off',
39+
'no-control-regex': 'warn',
4040
'no-debugger': 'warn',
4141
'no-delete-var': 'error',
4242
'no-dupe-class-members': 'error',

0 commit comments

Comments
 (0)