Skip to content

Commit c5deb45

Browse files
committed
Fix ESLint warnings.
1 parent 398b995 commit c5deb45

File tree

5 files changed

+20
-29
lines changed

5 files changed

+20
-29
lines changed

Lombiq.NodeJs.Extensions/eslint.config.cjs

Lines changed: 0 additions & 13 deletions
This file was deleted.

Lombiq.NodeJs.Extensions/scripts/eslint-msbuild-formatter.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
const { handleWarningObject, handleErrorObject } = require('./handle-error');
22

33
function numberOrOne(value) {
4-
const number = Number.parseInt(value);
4+
const number = Number.parseInt(value, 10);
55
return Number.isNaN(number) ? 1 : number;
66
}
77

Lombiq.NodeJs.Extensions/scripts/get-config.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ function getConfig({ directory, verbose }) {
3838
nodejsExtensionsConfig = JSON.parse(packageConfigJson)[configKeyInPackageJson] ?? {};
3939
logLine('succeeded.');
4040
}
41-
catch (_) {
41+
catch {
4242
logLine('failed.');
4343
}
4444

Lombiq.NodeJs.Extensions/scripts/lint-code.js

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,18 @@
11
const path = require('path');
22
const { ESLint } = require('eslint');
33

4-
const { formatter } = require('.nx/scripts/eslint-msbuild-formatter');
4+
const { formatter } = require('./eslint-msbuild-formatter');
55
const { handleErrorMessage } = require('./handle-error');
66

77
function getSourceType(filePath) {
88
const parts = filePath.split('.');
99
const extension = parts[parts.length - 1].toLowerCase();
1010

11-
return extension === 'mjs' ? 'module' :
12-
extension === 'cjs' ? 'commonjs' :
13-
undefined;
11+
switch (extension?.toLowerCase()) {
12+
case 'mjs': return 'module';
13+
case 'cjs': return 'commonjs';
14+
default: return undefined;
15+
}
1416
}
1517

1618
async function lintCode(code, id, firstRow = 1, overrideConfig = {}, formatterBeforeHandle = null) {
@@ -20,7 +22,7 @@ async function lintCode(code, id, firstRow = 1, overrideConfig = {}, formatterBe
2022

2123
const options = {
2224
cwd: path.dirname(filePath),
23-
overrideConfig: { ...overrideConfig }
25+
overrideConfig: { ...overrideConfig },
2426
};
2527

2628
if (!options.overrideConfig.languageOptions?.sourceType) {

Lombiq.NodeJs.Extensions/scripts/lint-markdown.js

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,16 @@ const fs = require('fs');
22
const path = require('path');
33
const process = require('process');
44

5-
const textlintPluginMarkdown = require("@textlint/textlint-plugin-markdown").default;
6-
const textLintFilterRuleComments = require("textlint-filter-rule-comments");
7-
const textLintRuleCommonMisspellings = require("textlint-rule-common-misspellings").default;
8-
const textLintRuleDoubledSpaces = require("textlint-rule-doubled-spaces").default;
9-
const textLintRuleMaxComma = require("textlint-rule-max-comma").default;
10-
const textLintRuleNoEmptySection = require("textlint-rule-no-empty-section");
11-
const textLintRuleNoTodo = require("textlint-rule-no-todo").default;
12-
const textLintRuleNoZeroWidthSpaces = require("textlint-rule-no-zero-width-spaces").default;
5+
/* eslint-disable import/no-unresolved -- False positive, they are in the package.json. */
6+
const textlintPluginMarkdown = require('@textlint/textlint-plugin-markdown').default;
7+
const textLintFilterRuleComments = require('textlint-filter-rule-comments');
8+
const textLintRuleCommonMisspellings = require('textlint-rule-common-misspellings').default;
9+
const textLintRuleDoubledSpaces = require('textlint-rule-doubled-spaces').default;
10+
const textLintRuleMaxComma = require('textlint-rule-max-comma').default;
11+
const textLintRuleNoEmptySection = require('textlint-rule-no-empty-section');
12+
const textLintRuleNoTodo = require('textlint-rule-no-todo').default;
13+
const textLintRuleNoZeroWidthSpaces = require('textlint-rule-no-zero-width-spaces').default;
14+
/* eslint-enable import/no-unresolved */
1315

1416
const findRecursively = require('./find-recursively');
1517
const { handleErrorObject, handleWarningObject } = require('./handle-error');
@@ -109,7 +111,7 @@ function newTextlintKernelOptions(config) {
109111
...config,
110112
plugins: [
111113
{
112-
pluginId: "markdown",
114+
pluginId: 'markdown',
113115
plugin: textlintPluginMarkdown,
114116
}
115117
],

0 commit comments

Comments
 (0)