Skip to content

Commit a3d17e5

Browse files
committed
refactor: drop migrate compat parserOptions rewrite
1 parent 5b003d7 commit a3d17e5

2 files changed

Lines changed: 2 additions & 28 deletions

File tree

src/migrate.ts

Lines changed: 1 addition & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -61,13 +61,6 @@ function buildMigratedEslintConfig(
6161
const globals = require('globals');
6262
6363
const legacyRules = ${rulesConfig};
64-
if (!Object.prototype.hasOwnProperty.call(
65-
legacyRules,
66-
'@typescript-eslint/no-floating-promises'
67-
)) {
68-
legacyRules['@typescript-eslint/no-floating-promises'] = 'off';
69-
}
70-
7164
const legacyEnv = ${envConfig};
7265
const legacyGlobals = Object.entries(legacyEnv).reduce((acc, [name, enabled]) => {
7366
if (!enabled || !globals[name]) {
@@ -76,27 +69,11 @@ const legacyGlobals = Object.entries(legacyEnv).reduce((acc, [name, enabled]) =>
7669
return { ...acc, ...globals[name] };
7770
}, {});
7871
79-
const compatConfig = mwtsConfig.map(config => {
80-
const parserOptions = config?.languageOptions?.parserOptions;
81-
if (!parserOptions) {
82-
return config;
83-
}
84-
const { project, projectService, ...nextParserOptions } = parserOptions;
85-
86-
return {
87-
...config,
88-
languageOptions: {
89-
...config.languageOptions,
90-
parserOptions: nextParserOptions,
91-
},
92-
};
93-
});
94-
9572
module.exports = [
9673
{
9774
ignores: ${ignoreList.trim()},
9875
},
99-
...compatConfig,
76+
...mwtsConfig,
10077
{
10178
languageOptions: {
10279
globals: legacyGlobals,

test/test-migrate.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,15 +39,12 @@ describe('migrate', () => {
3939
'utf8'
4040
);
4141
assert.ok(eslintConfig.includes("require('mwts/eslint.config.js')"));
42-
assert.ok(eslintConfig.includes('const { project, projectService'));
4342
assert.ok(
4443
eslintConfig.includes("ignores: ['dist/', '**/node_modules/',")
4544
);
4645
assert.ok(eslintConfig.includes("'packages/version'"));
4746
assert.ok(eslintConfig.includes("'site'"));
48-
assert.ok(
49-
eslintConfig.includes('@typescript-eslint/no-floating-promises')
50-
);
47+
assert.ok(eslintConfig.includes('...mwtsConfig'));
5148
assert.ok(eslintConfig.includes('no-control-regex'));
5249
assert.ok(eslintConfig.includes('const legacyEnv = {'));
5350
assert.ok(eslintConfig.includes('"jest": true'));

0 commit comments

Comments
 (0)