test: put common lint exceptions into config file
For lint exceptions that are universal or near universal for `test/common`, put the exceptions in a config file rather than disabling the ESLint rules at the top of each file. PR-URL: https://github.com/nodejs/node/pull/39358 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This commit is contained in:
parent
4011355d40
commit
c506660f32
3
test/common/.eslintrc.yaml
Normal file
3
test/common/.eslintrc.yaml
Normal file
@ -0,0 +1,3 @@
|
||||
rules:
|
||||
node-core/required-modules: off
|
||||
node-core/require-common-first: off
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
|
||||
const { Stream } = require('stream');
|
||||
|
@ -1,5 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
|
||||
'use strict';
|
||||
|
||||
const assert = require('assert');
|
||||
|
@ -1,5 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
|
||||
'use strict';
|
||||
|
||||
const assert = require('assert');
|
||||
|
@ -1,5 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
|
||||
'use strict';
|
||||
|
||||
require('./');
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
|
||||
const assert = require('assert');
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
const { Duplex } = require('stream');
|
||||
const assert = require('assert');
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
|
||||
const path = require('path');
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
import fixtures from './fixtures.js';
|
||||
|
||||
const {
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
const assert = require('assert');
|
||||
const util = require('util');
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
|
||||
// Hijack stdout and stderr
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
|
||||
// An HTTP/2 testing tool used to create mock frames for direct testing
|
||||
|
@ -19,7 +19,6 @@
|
||||
// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
|
||||
// USE OR OTHER DEALINGS IN THE SOFTWARE.
|
||||
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
/* eslint-disable node-core/crypto-check */
|
||||
'use strict';
|
||||
const process = global.process; // Some tests tamper with the process global.
|
||||
|
@ -1,5 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
|
||||
import { createRequire } from 'module';
|
||||
|
||||
const require = createRequire(import.meta.url);
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
|
||||
// Utilities for internet-related tests
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
|
||||
const assert = require('assert');
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
const assert = require('assert');
|
||||
const fs = require('fs');
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
|
||||
if (require.main !== module) {
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
/* eslint-disable node-core/crypto-check */
|
||||
|
||||
'use strict';
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
|
||||
const fs = require('fs');
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
const { internalBinding } = require('internal/test/binding');
|
||||
const { JSUDPWrap } = internalBinding('js_udp_wrap');
|
||||
|
@ -1,4 +1,3 @@
|
||||
/* eslint-disable node-core/require-common-first, node-core/required-modules */
|
||||
'use strict';
|
||||
|
||||
const assert = require('assert');
|
||||
|
@ -1,5 +1,3 @@
|
||||
/* eslint-disable node-core/required-modules,node-core/require-common-first */
|
||||
|
||||
'use strict';
|
||||
|
||||
const { runInThisContext } = require('vm');
|
||||
|
Loading…
x
Reference in New Issue
Block a user