Remove override of node/global-require rule for resources dir (#2970

)
graphql · Mar 13, 2021 · c581573 · c581573
1 parent f8155ac
commit c581573
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/.eslintrc.yml b/.eslintrc.yml
@@ -688,7 +688,6 @@ overrides:
     rules:
       node/no-unpublished-require: off
       node/no-sync: off
-      node/global-require: off
       import/no-extraneous-dependencies: [error, { devDependencies: true }]
       import/no-nodejs-modules: off
       import/no-commonjs: off

diff --git a/resources/build-npm.js b/resources/build-npm.js
@@ -55,7 +55,10 @@ function babelBuild(srcPath, options) {
 }
 
 function buildPackageJSON() {
-  const packageJSON = require('../package.json');
+  const packageJSON = JSON.parse(
+    fs.readFileSync(require.resolve('../package.json'), 'utf-8'),
+  );
+
   delete packageJSON.private;
   delete packageJSON.scripts;
   delete packageJSON.devDependencies;

diff --git a/resources/eslint-internal-rules/index.js b/resources/eslint-internal-rules/index.js
@@ -1,7 +1,9 @@
 'use strict';
 
+const noDirImport = require('./no-dir-import');
+
 module.exports = {
   rules: {
-    'no-dir-import': require('./no-dir-import'),
+    'no-dir-import': noDirImport,
   },
 };