Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 92f3b42

Browse filesBrowse files
JLHwungaduh95
JLHwung
authored and
aduh95
committed
tools: bump eslint to v10, babel to v8.0.0-rc.2
PR-URL: #61905 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
1 parent bab750d commit 92f3b42
Copy full SHA for 92f3b42

11 files changed

+389-530Lines changed: 389 additions & 530 deletions

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎doc/api/perf_hooks.md‎

Copy file name to clipboardExpand all lines: doc/api/perf_hooks.md
+2Lines changed: 2 additions & 0 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -2191,6 +2191,8 @@ const timedImport = performance.timerify(async (module) => {
21912191
await timedImport('some-module');
21922192
```
21932193

2194+
<!-- eslint-disable no-global-assign -->
2195+
21942196
```cjs
21952197
'use strict';
21962198
const {
Collapse file

‎doc/api/single-executable-applications.md‎

Copy file name to clipboardExpand all lines: doc/api/single-executable-applications.md
+2Lines changed: 2 additions & 0 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -412,6 +412,8 @@ To load modules from the file system in the injected main script, users can
412412
create a `require` function that can load from the file system using
413413
`module.createRequire()`. For example, in a CommonJS entry point:
414414

415+
<!-- eslint-disable no-global-assign -->
416+
415417
```js
416418
const { createRequire } = require('node:module');
417419
require = createRequire(__filename);
Collapse file

‎eslint.config.mjs‎

Copy file name to clipboardExpand all lines: eslint.config.mjs
+4-4Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -85,10 +85,9 @@ export default [
8585
js.configs.recommended,
8686
jsdoc.configs['flat/recommended'],
8787
{
88-
files: ['**/*.{js,cjs}'],
88+
files: ['**/*.js'],
8989
languageOptions: {
90-
// The default is `commonjs` but it's not supported by the Babel parser.
91-
sourceType: 'script',
90+
sourceType: 'commonjs',
9291
},
9392
},
9493
{
@@ -101,7 +100,6 @@ export default [
101100
parser: babelEslintParser,
102101
parserOptions: {
103102
babelOptions: {
104-
parserOpts: { createImportExpressions: true },
105103
plugins: [
106104
babelPluginSyntaxImportSource,
107105
],
@@ -229,6 +227,7 @@ export default [
229227
...noRestrictedSyntaxCommonLib,
230228
],
231229
'no-self-compare': 'error',
230+
'no-shadow-restricted-names': ['error', { reportGlobalThis: false }],
232231
'no-template-curly-in-string': 'error',
233232
'no-throw-literal': 'error',
234233
'no-undef': ['error', { typeof: true }],
@@ -256,6 +255,7 @@ export default [
256255

257256
// ESLint recommended rules that we disable.
258257
'no-inner-declarations': 'off',
258+
'no-useless-assignment': 'off',
259259

260260
// JSDoc rules.
261261
'jsdoc/require-jsdoc': 'off',
Collapse file

‎lib/internal/modules/cjs/loader.js‎

Copy file name to clipboardExpand all lines: lib/internal/modules/cjs/loader.js
+1-3Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1798,8 +1798,6 @@ Module.prototype._compile = function(content, filename, format) {
17981798
}
17991799
}
18001800

1801-
let redirects;
1802-
18031801
let compiledWrapper;
18041802
if (format !== 'module') {
18051803
const result = wrapSafe(filename, content, this, format);
@@ -1815,7 +1813,7 @@ Module.prototype._compile = function(content, filename, format) {
18151813
}
18161814

18171815
const dirname = path.dirname(filename);
1818-
const require = makeRequireFunction(this, redirects);
1816+
const require = makeRequireFunction(this);
18191817
let result;
18201818
const exports = this.exports;
18211819
const thisValue = exports;
Collapse file

‎test/common/sea.js‎

Copy file name to clipboardExpand all lines: test/common/sea.js
+5-5Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ function generateSEA(fixtureDir, options = {}) {
154154
} catch (e) {
155155
const message = `Cannot copy ${process.execPath} to ${outputFile}: ${inspect(e)}`;
156156
if (verifyWorkflow) {
157-
throw new Error(message);
157+
throw new Error(message, { cause: e });
158158
}
159159
common.skip(message);
160160
}
@@ -192,7 +192,7 @@ function generateSEA(fixtureDir, options = {}) {
192192
} catch (e) {
193193
const message = `Cannot inject ${seaPrepBlob} into ${outputFile}: ${inspect(e)}`;
194194
if (verifyWorkflow) {
195-
throw new Error(message);
195+
throw new Error(message, { cause: e });
196196
}
197197
common.skip(message);
198198
}
@@ -210,7 +210,7 @@ function signSEA(targetExecutable, verifyWorkflow = false) {
210210
} catch (e) {
211211
const message = `Cannot sign ${targetExecutable}: ${inspect(e)}`;
212212
if (verifyWorkflow) {
213-
throw new Error(message);
213+
throw new Error(message, { cause: e });
214214
}
215215
common.skip(message);
216216
}
@@ -221,7 +221,7 @@ function signSEA(targetExecutable, verifyWorkflow = false) {
221221
} catch (e) {
222222
const message = `Cannot find signtool: ${inspect(e)}`;
223223
if (verifyWorkflow) {
224-
throw new Error(message);
224+
throw new Error(message, { cause: e });
225225
}
226226
common.skip(message);
227227
}
@@ -232,7 +232,7 @@ function signSEA(targetExecutable, verifyWorkflow = false) {
232232
} catch (e) {
233233
const message = `Cannot sign ${targetExecutable}: ${inspect(e)}\n${stderr}`;
234234
if (verifyWorkflow) {
235-
throw new Error(message);
235+
throw new Error(message, { cause: e });
236236
}
237237
common.skip(message);
238238
}
Collapse file

‎test/es-module/test-esm-detect-ambiguous.mjs‎

Copy file name to clipboardExpand all lines: test/es-module/test-esm-detect-ambiguous.mjs
+1Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,7 @@ describe('Module syntax detection', { concurrency: !process.env.TEST_PARALLEL },
147147
}
148148

149149
it('should not hint wrong format in resolve hook', async () => {
150+
// eslint-disable-next-line no-unassigned-vars
150151
let writeSync;
151152
const { stdout, stderr, code, signal } = await spawnPromisified(process.execPath, [
152153
'--no-warnings',
Collapse file

‎tools/eslint-rules/prefer-optional-chaining.js‎

Copy file name to clipboardExpand all lines: tools/eslint-rules/prefer-optional-chaining.js
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ module.exports = {
1515
},
1616

1717
create(context) {
18-
const sourceCode = context.getSourceCode();
18+
const sourceCode = context.sourceCode;
1919

2020
// Helper function: Checks if two nodes have identical tokens
2121
function equalTokens(left, right) {
Collapse file

‎tools/eslint-rules/require-common-first.js‎

Copy file name to clipboardExpand all lines: tools/eslint-rules/require-common-first.js
+7-6Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -49,12 +49,13 @@ module.exports = {
4949
// The common module should be loaded in the first place.
5050
const notLoadedFirst = foundModules.indexOf(requiredModule) !== 0;
5151
if (notLoadedFirst) {
52-
context.report(
53-
node,
54-
'Mandatory module "{{moduleName}}" must be loaded ' +
55-
'before any other modules.',
56-
{ moduleName: requiredModule },
57-
);
52+
context.report({
53+
node: node.body[0] ?? node,
54+
message:
55+
'Mandatory module "{{moduleName}}" must be loaded ' +
56+
'before any other modules.',
57+
data: { moduleName: requiredModule },
58+
});
5859
}
5960
},
6061
};
Collapse file

‎tools/eslint-rules/required-modules.js‎

Copy file name to clipboardExpand all lines: tools/eslint-rules/required-modules.js
+5-5Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -66,11 +66,11 @@ module.exports = {
6666
([module]) => foundModules.indexOf(module) === -1,
6767
);
6868
missingModules.forEach(([moduleName]) => {
69-
context.report(
70-
node,
71-
'Mandatory module "{{moduleName}}" must be loaded.',
72-
{ moduleName: moduleName },
73-
);
69+
context.report({
70+
node: node.body[0] ?? node,
71+
message: 'Mandatory module "{{moduleName}}" must be loaded.',
72+
data: { moduleName: moduleName },
73+
});
7474
});
7575
}
7676
},

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.