Skip to content

Commit 83f7647

Browse files
committed
lint:fix
1 parent 55571e2 commit 83f7647

File tree

13 files changed

+67
-60
lines changed

13 files changed

+67
-60
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ rules in templates can be disabled with eslint directives with mustache or html
172172
![gjs logo](/docs/svgs/gjs.svg) Set in the `recommended-gjs` [configuration](https://github.com/ember-cli/eslint-plugin-ember#-configurations).\
173173
![gts logo](/docs/svgs/gts.svg) Set in the `recommended-gts` [configuration](https://github.com/ember-cli/eslint-plugin-ember#-configurations).\
174174
🔧 Automatically fixable by the [`--fix` CLI option](https://eslint.org/docs/user-guide/command-line-interface#--fix).\
175-
💡 Manually fixable by [editor suggestions](https://eslint.org/docs/developer-guide/working-with-rules#providing-suggestions).
175+
💡 Manually fixable by [editor suggestions](https://eslint.org/docs/latest/use/core-concepts#rule-suggestions).
176176

177177
### Components
178178

docs/rules/no-unused-services.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# ember/no-unused-services
22

3-
💡 This rule is manually fixable by [editor suggestions](https://eslint.org/docs/developer-guide/working-with-rules#providing-suggestions).
3+
💡 This rule is manually fixable by [editor suggestions](https://eslint.org/docs/latest/use/core-concepts#rule-suggestions).
44

55
<!-- end auto-generated rule header -->
66

docs/rules/route-path-style.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# ember/route-path-style
22

3-
💡 This rule is manually fixable by [editor suggestions](https://eslint.org/docs/developer-guide/working-with-rules#providing-suggestions).
3+
💡 This rule is manually fixable by [editor suggestions](https://eslint.org/docs/latest/use/core-concepts#rule-suggestions).
44

55
<!-- end auto-generated rule header -->
66

eslint.config.js

Lines changed: 21 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -32,23 +32,10 @@ module.exports = [
3232
eslintPluginN.configs['flat/recommended'],
3333
eslintPluginUnicorn.configs['flat/recommended'],
3434

35-
{
36-
ignores: [
37-
'coverage/',
38-
'node_modules/',
39-
'lib/recommended-rules.js',
40-
'lib/recommended-rules-gjs.js',
41-
'lib/recommended-rules-gts.js',
42-
'tests/__snapshots__/',
43-
44-
// # Contains <template> in js markdown
45-
'docs/rules/no-empty-glimmer-component-classes.md',
46-
'docs/rules/template-no-let-reference.md',
47-
],
48-
},
4935
{
5036
languageOptions: {
5137
parser: babelEslintParser,
38+
sourceType: 'script',
5239
parserOptions: {
5340
ecmaVersion: 2022,
5441
sourceType: 'script',
@@ -152,6 +139,8 @@ module.exports = [
152139
'import/no-webpack-loader-syntax': 'error',
153140
'import/unambiguous': 'error',
154141

142+
'n/no-unsupported-features/node-builtins': 'off',
143+
155144
// Unicorn rules:
156145
'unicorn/no-array-callback-reference': 'off',
157146
'unicorn/no-array-method-this-argument': 'off',
@@ -183,6 +172,7 @@ module.exports = [
183172
// Markdown code samples in documentation:
184173
files: ['**/*.md/*.js'],
185174
languageOptions: {
175+
sourceType: 'module',
186176
parserOptions: {
187177
sourceType: 'module',
188178
ecmaFeatures: { legacyDecorators: true },
@@ -232,6 +222,23 @@ module.exports = [
232222
'import/no-named-as-default-member': 'off',
233223
'import/no-unresalved': 'off',
234224
'import/no-missing-import': 'off',
225+
226+
// vite config format does not match regex
227+
'filenames/match-regex': 'off',
235228
},
236229
},
230+
{
231+
ignores: [
232+
'coverage/',
233+
'node_modules/',
234+
'lib/recommended-rules.js',
235+
'lib/recommended-rules-gjs.js',
236+
'lib/recommended-rules-gts.js',
237+
'tests/__snapshots__/',
238+
239+
// # Contains <template> in js markdown
240+
'docs/rules/no-empty-glimmer-component-classes.md',
241+
'docs/rules/template-no-let-reference.md',
242+
],
243+
},
237244
];

lib/rules/no-get.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ module.exports = {
224224
);
225225
}
226226

227-
let proxyObjects = [];
227+
const proxyObjects = [];
228228
let currentClassWithUnknownPropertyMethod = null;
229229

230230
let importedGetName;

package.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,8 @@
4444
"lint:eslint-docs": "npm-run-all \"update:eslint-docs -- --check\"",
4545
"lint:eslint-docs:fix": "npm-run-all \"update:eslint-docs\"",
4646
"lint:fix": "npm-run-all \"lint:*:fix\"",
47-
"lint:js": "eslint --cache .",
48-
"lint:js:fix": "npm-run-all \"lint:js -- --fix\"",
47+
"lint:js": "eslint .",
48+
"lint:js:fix": "pnpm lint:js --fix",
4949
"lint:package-json": "npmPkgJsonLint .",
5050
"lint:package-json-sorting": "sort-package-json --check",
5151
"lint:package-json-sorting:fix": "sort-package-json package.json",
@@ -89,7 +89,7 @@
8989
"eslint-plugin-filenames": "^1.3.2",
9090
"eslint-plugin-import": "^2.29.0",
9191
"eslint-plugin-markdown": "^3.0.1",
92-
"eslint-plugin-n": "^16.2.0",
92+
"eslint-plugin-n": "^17.11.1",
9393
"eslint-plugin-prettier": "^5.0.1",
9494
"eslint-plugin-unicorn": "^51.0.0",
9595
"eslint-remote-tester": "^3.0.1",
@@ -116,10 +116,10 @@
116116
"optional": true
117117
}
118118
},
119+
"packageManager": "[email protected]",
119120
"engines": {
120121
"node": "18.* || 20.* || >= 21"
121122
},
122-
"packageManager": "[email protected]",
123123
"publishConfig": {
124124
"registry": "https://registry.npmjs.org"
125125
},

pnpm-lock.yaml

Lines changed: 31 additions & 20 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tests/config-setup.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ describe('config setup is correct', function () {
1111
const filePath = path.join(__dirname, '..', 'lib', 'config');
1212
const files = readdirSync(filePath);
1313

14-
// eslint-disable-next-line jest/prefer-strict-equal
1514
expect(CONFIG_NAMES).toEqual(
1615
files.filter((file) => !file.startsWith('.')).map((file) => file.replace('.js', ''))
1716
);

tests/lib/rules-preprocessor/gjs-gts-parser-test.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -491,7 +491,6 @@ describe('template-vars', () => {
491491
for (const scenario of valid) {
492492
const { code, filename, parser } = scenario;
493493

494-
// eslint-disable-next-line jest/valid-title
495494
it(code, async () => {
496495
const eslint = initESLint(parser);
497496
const results = await eslint.lintText(code, {
@@ -511,7 +510,6 @@ describe('template-vars', () => {
511510
for (const scenario of invalid) {
512511
const { code, filename, errors, parser } = scenario;
513512

514-
// eslint-disable-next-line jest/valid-title
515513
it(code, async () => {
516514
const eslint = initESLint(parser);
517515
const results = await eslint.lintText(code, {

tests/lib/utils/property-order-test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -496,7 +496,7 @@ describe('reportUnorderedProperties', () => {
496496
undefined,
497497
importControllerName
498498
);
499-
expect(context.report).toHaveBeenCalled(); // eslint-disable-line jest/prefer-called-with
499+
expect(context.report).toHaveBeenCalled();
500500
});
501501
});
502502

@@ -558,7 +558,7 @@ describe('reportUnorderedProperties', () => {
558558
undefined,
559559
importControllerName
560560
);
561-
expect(context.report).toHaveBeenCalled(); // eslint-disable-line jest/prefer-called-with
561+
expect(context.report).toHaveBeenCalled();
562562
});
563563
});
564564
});

0 commit comments

Comments
 (0)