GeezerVison is showing github.com — tools: enforce trailing commas in `test/sequential` · nodejs/node@eebd732
File treeExpand file treeCollapse file tree Open diff view settings Some content is hiddenLarge Commits have some content hidden by default. Use the searchbox below for content that may be hidden. Dismiss bannerExpand file treeCollapse file tree Open diff view settings Collapse file Copy file name to clipboardExpand all lines: test/eslint.config_partial.mjs +17-1Lines changed: 17 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -148,7 +148,6 @@ export default [ | 148 | 148 | files: [ | 149 | 149 | 'test/es-module/**/*.{js,mjs}', | 150 | 150 | 'test/parallel/**/*.{js,mjs}', | 151 | | - 'test/sequential/**/*.{js,mjs}', | 152 | 151 | ], | 153 | 152 | rules: { | 154 | 153 | '@stylistic/js/comma-dangle': [ | @@ -163,6 +162,23 @@ export default [ | 163 | 162 | ], | 164 | 163 | }, | 165 | 164 | }, | | 165 | + { | | 166 | + files: [ | | 167 | + 'test/sequential/**/*.{js,mjs}', | | 168 | + ], | | 169 | + rules: { | | 170 | + '@stylistic/js/comma-dangle': [ | | 171 | + 'error', | | 172 | + { | | 173 | + arrays: 'always-multiline', | | 174 | + exports: 'always-multiline', | | 175 | + functions: 'only-multiline', | | 176 | + imports: 'always-multiline', | | 177 | + objects: 'always-multiline', | | 178 | + }, | | 179 | + ], | | 180 | + }, | | 181 | + }, | 166 | 182 | { | 167 | 183 | files: [ | 168 | 184 | 'test/{common,fixtures,wpt}/**/*.{js,mjs,cjs}', | |
Collapse file Copy file name to clipboardExpand all lines: test/sequential/test-child-process-execsync.js +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -156,5 +156,5 @@ const args = [ | 156 | 156 |
| 157 | 157 | // Verify the shell option works properly | 158 | 158 | execFileSync(`"${common.isWindows ? process.execPath : '$NODE'}"`, [], { | 159 | | - encoding: 'utf8', shell: true, env | | 159 | + encoding: 'utf8', shell: true, env, | 160 | 160 | }); |
Collapse file Copy file name to clipboardExpand all lines: test/sequential/test-child-process-exit.js +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -40,7 +40,7 @@ if (gen === maxGen) { | 40 | 40 | } | 41 | 41 |
| 42 | 42 | const child = ch.spawn(process.execPath, [__filename, gen + 1], { | 43 | | - stdio: [ 'ignore', 'pipe', 'ignore' ] | | 43 | + stdio: [ 'ignore', 'pipe', 'ignore' ], | 44 | 44 | }); | 45 | 45 | assert.ok(!child.stdin); | 46 | 46 | assert.ok(child.stdout); | |
Collapse file Copy file name to clipboardExpand all lines: test/sequential/test-cli-syntax-require.js +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -28,7 +28,7 @@ const syntaxErrorRE = /^SyntaxError: \b/m; | 28 | 28 |
| 29 | 29 | // stderr should include the filename | 30 | 30 | assert(output.startsWith(file), `${output} starts with ${file}`); | 31 | | - } | | 31 | + }, | 32 | 32 | }); | 33 | 33 | }); | 34 | 34 | }); |
Collapse file Copy file name to clipboardExpand all lines: test/sequential/test-cluster-inspect-brk.js +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -14,7 +14,7 @@ if (cluster.isPrimary) { | 14 | 14 |
| 15 | 15 | cluster.setupPrimary({ | 16 | 16 | execArgv: execArgv, | 17 | | - stdio: ['pipe', 'pipe', 'pipe', 'ipc', 'pipe'] | | 17 | + stdio: ['pipe', 'pipe', 'pipe', 'ipc', 'pipe'], | 18 | 18 | }); | 19 | 19 |
| 20 | 20 | const worker = cluster.fork(); | |
Collapse file Copy file name to clipboardExpand all lines: test/sequential/test-cpu-prof-default.js +2-2Lines changed: 2 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -15,7 +15,7 @@ const { spawnSync } = require('child_process'); | 15 | 15 | const tmpdir = require('../common/tmpdir'); | 16 | 16 | const { | 17 | 17 | getCpuProfiles, | 18 | | - env | | 18 | + env, | 19 | 19 | } = require('../common/cpu-prof'); | 20 | 20 |
| 21 | 21 | { | @@ -25,7 +25,7 @@ const { | 25 | 25 | fixtures.path('workload', 'fibonacci.js'), | 26 | 26 | ], { | 27 | 27 | cwd: tmpdir.path, | 28 | | - env | | 28 | + env, | 29 | 29 | }); | 30 | 30 | if (output.status !== 0) { | 31 | 31 | console.log(output.stderr.toString()); | |
Collapse file Copy file name to clipboardExpand all lines: test/sequential/test-cpu-prof-dir-absolute.js +2-2Lines changed: 2 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -15,7 +15,7 @@ const { | 15 | 15 | getCpuProfiles, | 16 | 16 | kCpuProfInterval, | 17 | 17 | env, | 18 | | - verifyFrames | | 18 | + verifyFrames, | 19 | 19 | } = require('../common/cpu-prof'); | 20 | 20 |
| 21 | 21 | // relative --cpu-prof-dir | @@ -31,7 +31,7 @@ const { | 31 | 31 | fixtures.path('workload', 'fibonacci.js'), | 32 | 32 | ], { | 33 | 33 | cwd: tmpdir.path, | 34 | | - env | | 34 | + env, | 35 | 35 | }); | 36 | 36 | if (output.status !== 0) { | 37 | 37 | console.log(output.stderr.toString()); | |
Collapse file Copy file name to clipboardExpand all lines: test/sequential/test-cpu-prof-dir-and-name.js +2-2Lines changed: 2 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -16,7 +16,7 @@ const { | 16 | 16 | getCpuProfiles, | 17 | 17 | kCpuProfInterval, | 18 | 18 | env, | 19 | | - verifyFrames | | 19 | + verifyFrames, | 20 | 20 | } = require('../common/cpu-prof'); | 21 | 21 |
| 22 | 22 | { | @@ -34,7 +34,7 @@ const { | 34 | 34 | fixtures.path('workload', 'fibonacci.js'), | 35 | 35 | ], { | 36 | 36 | cwd: tmpdir.path, | 37 | | - env | | 37 | + env, | 38 | 38 | }); | 39 | 39 | if (output.status !== 0) { | 40 | 40 | console.log(output.stderr.toString()); | |
Collapse file Copy file name to clipboardExpand all lines: test/sequential/test-cpu-prof-dir-relative.js +2-2Lines changed: 2 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -15,7 +15,7 @@ const { | 15 | 15 | getCpuProfiles, | 16 | 16 | kCpuProfInterval, | 17 | 17 | env, | 18 | | - verifyFrames | | 18 | + verifyFrames, | 19 | 19 | } = require('../common/cpu-prof'); | 20 | 20 |
| 21 | 21 | // relative --cpu-prof-dir | @@ -30,7 +30,7 @@ const { | 30 | 30 | fixtures.path('workload', 'fibonacci.js'), | 31 | 31 | ], { | 32 | 32 | cwd: tmpdir.path, | 33 | | - env | | 33 | + env, | 34 | 34 | }); | 35 | 35 | if (output.status !== 0) { | 36 | 36 | console.log(output.stderr.toString()); | |
Collapse file Copy file name to clipboardExpand all lines: test/sequential/test-cpu-prof-dir-worker.js +2-2Lines changed: 2 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -15,7 +15,7 @@ const { | 15 | 15 | getCpuProfiles, | 16 | 16 | kCpuProfInterval, | 17 | 17 | env, | 18 | | - getFrames | | 18 | + getFrames, | 19 | 19 | } = require('../common/cpu-prof'); | 20 | 20 |
| 21 | 21 | // --cpu-prof-dir with worker | @@ -30,7 +30,7 @@ const { | 30 | 30 | fixtures.path('workload', 'fibonacci-worker.js'), | 31 | 31 | ], { | 32 | 32 | cwd: tmpdir.path, | 33 | | - env | | 33 | + env, | 34 | 34 | }); | 35 | 35 | if (output.status !== 0) { | 36 | 36 | console.log(output.stderr.toString()); | |
|