Skip to content

Commit 57b264f

Browse files
committed
feat(eslint-config): update next and nest override patterns
1 parent af05b43 commit 57b264f

File tree

3 files changed

+11
-8
lines changed

3 files changed

+11
-8
lines changed

packages/eslint-config/src/base/config.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,10 @@ export const config: ConfigArray = tsEslint.config(
4545
unicorn,
4646
},
4747
languageOptions: {
48-
globals: globals.builtin,
48+
globals: {
49+
...globals.builtin,
50+
...globals.node,
51+
},
4952
parser: tsEslintParser,
5053
parserOptions: {
5154
ecmaVersion: 'latest',

packages/eslint-config/src/nestjs/config.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,13 @@ const compat = new FlatCompat({
77
export const [config] = compat.config({
88
overrides: [
99
{
10-
files: ['**/src/**/models/**/*.ts'],
10+
files: ['src/**/models/**/*.ts'],
1111
rules: {
1212
'import/no-cycle': 'off', // Allow circular dependencies in models as they might reference each other
1313
},
1414
},
1515
{
16-
files: ['**/src/**/*.ts'],
16+
files: ['src/**/*.ts'],
1717
rules: {
1818
'class-methods-use-this': 'off', // Allow class methods that don't use `this` keyword
1919
'no-empty-function': ['error', { allow: ['constructors'] }], // Allow empty constructors for Nest classes

packages/eslint-config/src/nextjs/config.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,13 @@ export const [config] = compat.config({
1414
},
1515
overrides: [
1616
{
17-
files: ['**/next-env.d.ts'],
17+
files: ['next-env.d.ts'],
1818
rules: {
1919
'unicorn/prevent-abbreviations': 'off', // next-env.d.ts is a Next.js convention
2020
},
2121
},
2222
{
23-
files: ['**/src/app/**/*.ts', '**/src/app/**/*.tsx', '**/src/pages/**/*.ts', '**/src/pages/**/*.tsx'],
23+
files: ['src/app/**/*.ts', 'src/app/**/*.tsx', 'src/pages/**/*.ts', 'src/pages/**/*.tsx'],
2424
rules: {
2525
'react/function-component-definition': [
2626
// Allow components as function declarations in Next.js App and Page components for shorthand default exports
@@ -33,19 +33,19 @@ export const [config] = compat.config({
3333
},
3434
},
3535
{
36-
files: ['**/src/app/**/layout.tsx', '**/src/**/layouts/**/*.tsx'],
36+
files: ['app/**/layout.tsx', 'src/**/layouts/**/*.tsx'],
3737
rules: {
3838
'@next/next/no-head-element': 'off', // Allow <head/> element in Next.js App layouts
3939
},
4040
},
4141
{
42-
files: ['**/src/**/layouts/**/*.tsx', '**/src/pages/_app.tsx', '**/src/pages/_document.tsx'],
42+
files: ['src/**/layouts/**/*.tsx', 'rc/pages/_app.tsx', 'src/pages/_document.tsx'],
4343
rules: {
4444
'react/jsx-props-no-spreading': 'off', // Allow spreading props in Next.js App layouts, _app.tsx and _document.tsx
4545
},
4646
},
4747
{
48-
files: ['**/src/**/actions.ts'],
48+
files: ['src/**/actions.ts'],
4949
rules: {
5050
'@typescript-eslint/require-await': 'off', // Next.js Server Actions must be async but don't need to await
5151
},

0 commit comments

Comments
 (0)