allow i18n scripts to take paths as parameters

Author: alexcjohnson

scripts/combineTranslationKeys.js

@@ -1,32 +1,45 @@
 import path from 'path';
 import fs from 'fs';
 
+// generalize so we can use this script in other es6 repos
+// so you can call:
+//   combineTranslationKeys <inputPath> <inputPath> <inputPath> ... <outputPath>
+
 const pathToCombinedTranslationKeys = path.join(
   __dirname,
   './translationKeys/combined-translation-keys.txt'
 );
 
-const plotlyJS = {
-  repository: 'plotly.js',
-  path: path.join(
-    __dirname,
-    '../node_modules/plotly.js/dist/translation-keys.txt'
-  ),
-};
+const plotlyJS = path.join(
+  __dirname,
+  '../node_modules/plotly.js/dist/translation-keys.txt'
+);
+
+const editor = path.join(__dirname, './translationKeys/translation-keys.txt');
+
+const argvLen = process.argv.length;
+const minHasPaths = 4;
+
+const hasPaths = argvLen >= minHasPaths;
 
-const editor = {
-  repository: 'react-plotly.js-editor',
-  path: path.join(__dirname, './translationKeys/translation-keys.txt'),
-};
+const inputPaths = hasPaths
+  ? process.argv.slice(2, argvLen - 1)
+  : [plotlyJS, editor];
+
+const outputPath = hasPaths
+  ? process.argv[argvLen - 1]
+  : pathToCombinedTranslationKeys;
 
 combineTranslationKeys();
 
 function combineTranslationKeys() {
   const dict = {};
   let maxLen = 0;
 
-  [plotlyJS, editor].forEach(file => {
-    const lines = fs.readFileSync(file.path, 'utf-8').split(/\r?\n/);
+  inputPaths.forEach(inputPath => {
+    const lines = fs.readFileSync(inputPath, 'utf-8').split(/\r?\n/);
+
+    const repository = getRepository(inputPath);
 
     lines.forEach(line => {
       const splitString = line.split(/\/\//);
@@ -35,9 +48,9 @@ function combineTranslationKeys() {
       maxLen = Math.max(maxLen, stringToTranslate.length);
 
       if (!dict[stringToTranslate]) {
-        dict[stringToTranslate] = ' // ' + file.repository + ': ' + source;
+        dict[stringToTranslate] = ' // ' + repository + ': ' + source;
       } else {
-        dict[stringToTranslate] += ` && ${file.repository}: ${source}`;
+        dict[stringToTranslate] += ` && ${repository}: ${source}`;
       }
     });
   });
@@ -47,10 +60,16 @@ function combineTranslationKeys() {
     .map(k => k + spaces(maxLen - k.length) + dict[k])
     .join('\n');
 
-  fs.writeFile(pathToCombinedTranslationKeys, strings);
-  console.log(
-    `combined translation keys were written to: ${pathToCombinedTranslationKeys}`
-  );
+  fs.writeFile(outputPath, strings);
+  console.log(`combined translation keys were written to: ${outputPath}`);
+}
+
+function getRepository(inputPath) {
+  var dir = path.dirname(inputPath);
+  if (fs.existsSync(path.join(dir, 'package.json'))) {
+    return path.basename(dir);
+  }
+  return getRepository(dir);
 }
 
 function spaces(len) {

scripts/findTranslationKeys.js

@@ -4,10 +4,16 @@ import fs from 'fs';
 import glob from 'glob';
 import path from 'path';
 
-const pathToSrc = path.join(__dirname, '../src');
+// generalize so we can use this script in other es6 repos
+// so you can call:
+//   findTranslationKeys <srcPath> <outputPath>
+const pathToSrc = process.argv[2] || path.join(__dirname, '../src');
 const srcGlob = path.join(pathToSrc, '**/*.js');
 
-const localizeRE = /(^|[\.])(_|localize)$/;
+const pathToTranslationKeys = process.argv[3] || path.join(
+  __dirname,
+  './translationKeys/translation-keys.txt'
+);
 
 findLocaleStrings();
 
@@ -84,10 +90,6 @@ function findLocaleStrings() {
       .map(k => k + spaces(maxLen - k.length) + '  // ' + dict[k])
       .join('\n');
 
-    const pathToTranslationKeys = path.join(
-      __dirname,
-      './translationKeys/translation-keys.txt'
-    );
     fs.writeFile(pathToTranslationKeys, strings);
     console.log(`translation keys were written to: ${pathToTranslationKeys}`);
   });

scripts/makeArrows.js

@@ -1,12 +1,19 @@
 import path from 'path';
 import fs from 'fs';
 
-const pathToCombinedTranslationKeys = path.join(
+// generalize so we can use this script in other repos
+// so you can call:
+//   makeArrows <srcPath> <outputPath>
+
+const pathToCombinedTranslationKeys = process.argv[2] || path.join(
   __dirname,
   './translationKeys/combined-translation-keys.txt'
 );
 
-const pathToArrowsOut = path.join(__dirname, '../src/locales/xx.js');
+const pathToArrowsOut = process.argv[3] || path.join(
+  __dirname,
+  '../src/locales/xx.js'
+);
 
 const wordRE = /^[A-Za-z]+$/;