build: rename dist directory to match 'graphql-js' (#683)

Author: IvanGoncharov

.eslintignore

@@ -1,7 +1,7 @@
 # Copied from '.gitignore', please keep it in sync.
 /node_modules
 /coverage
-/dist
+/npmDist
 
 # Ignore TS files inside integration test
 /integrationTests/ts/*.ts

.gitignore

@@ -7,4 +7,4 @@
 
 /node_modules
 /coverage
-/dist
+/npmDist

.prettierignore

@@ -1,4 +1,4 @@
 # Copied from '.gitignore', please keep it in sync.
 /node_modules
 /coverage
-/dist
+/npmDist

cspell.json

@@ -5,7 +5,7 @@
     ".eslintcache",
     "node_modules",
     "coverage",
-    "dist",
+    "npmDist",
 
     // Excluded from spelling check
     "cspell.json",

integrationTests/integration-test.js

@@ -19,7 +19,7 @@ describe('Integration Tests', () => {
   fs.rmdirSync(tmpDir, { recursive: true });
   fs.mkdirSync(tmpDir);
 
-  const distDir = path.resolve('./dist');
+  const distDir = path.resolve('./npmDist');
   exec(`npm pack ${distDir} && cp express-graphql-*.tgz express-graphql.tgz`, {
     cwd: tmpDir,
   });

resources/build.js

@@ -12,15 +12,15 @@ const tsConfig = require('../tsconfig.json');
 const {
   transformLoadFileStaticallyFromNPM,
 } = require('./load-statically-from-npm');
-const { rmdirRecursive, readdirRecursive, showStats } = require('./utils');
+const { rmdirRecursive, readdirRecursive, showDirStats } = require('./utils');
 
 if (require.main === module) {
-  rmdirRecursive('./dist');
-  fs.mkdirSync('./dist');
+  rmdirRecursive('./npmDist');
+  fs.mkdirSync('./npmDist');
 
   const srcFiles = readdirRecursive('./src', { ignoreDir: /^__.*__$/ });
   const { options } = ts.convertCompilerOptionsFromJson(
-    tsConfig.compilerOptions,
+    { ...tsConfig.compilerOptions, outDir: 'npmDist' },
     process.cwd(),
   );
   const program = ts.createProgram({
@@ -30,16 +30,19 @@ if (require.main === module) {
   program.emit(undefined, undefined, undefined, undefined, {
     after: [transformLoadFileStaticallyFromNPM],
   });
-  downlevel('./dist', './dist/ts3.4');
+  downlevel('./npmDist', './npmDist/ts3.4');
 
-  fs.copyFileSync('./LICENSE', './dist/LICENSE');
-  fs.copyFileSync('./README.md', './dist/README.md');
+  fs.copyFileSync('./LICENSE', './npmDist/LICENSE');
+  fs.copyFileSync('./README.md', './npmDist/README.md');
 
   // Should be done as the last step so only valid packages can be published
   const packageJSON = buildPackageJSON();
-  fs.writeFileSync('./dist/package.json', JSON.stringify(packageJSON, null, 2));
+  fs.writeFileSync(
+    './npmDist/package.json',
+    JSON.stringify(packageJSON, null, 2),
+  );
 
-  showStats();
+  showDirStats('./npmDist');
 }
 
 function buildPackageJSON() {

resources/utils.js

@@ -67,19 +67,19 @@ function readdirRecursive(dirPath, opts = {}) {
   return result;
 }
 
-function showStats() {
+function showDirStats(dirPath) {
   const fileTypes = {};
   let totalSize = 0;
 
-  for (const filepath of readdirRecursive('./dist')) {
+  for (const filepath of readdirRecursive(dirPath)) {
     const name = filepath.split(path.sep).pop();
     const [base, ...splitExt] = name.split('.');
     const ext = splitExt.join('.');
 
     const filetype = ext ? '*.' + ext : base;
     fileTypes[filetype] = fileTypes[filetype] || { filepaths: [], size: 0 };
 
-    const { size } = fs.lstatSync(path.join('./dist', filepath));
+    const { size } = fs.lstatSync(path.join(dirPath, filepath));
     totalSize += size;
     fileTypes[filetype].size += size;
     fileTypes[filetype].filepaths.push(filepath);
@@ -118,5 +118,5 @@ module.exports = {
   execAsync,
   rmdirRecursive,
   readdirRecursive,
-  showStats,
+  showDirStats,
 };

tsconfig.json

@@ -10,7 +10,6 @@
     "noImplicitThis": true,
     "strictNullChecks": true,
     "strictFunctionTypes": true,
-    "outDir": "dist",
     "declaration": true
   }
 }