adds back the explicit-require option (#232)

thanks to ismusidhu in #205

Author: Sunil Pai

README.md

@@ -198,6 +198,10 @@ through jscodeshift's `printOptions` command line argument
 npx react-codemod <transform> <path> --jscodeshift="--printOptions='{\"quote\":\"double\"}'"
 ```
 
+### `explicit-require=false`
+
+If you're not explicitly importing React in your files (eg: if you're loading React with a script tag), you should add `--explicit-require=false`. 
+
 ### Support and Contributing
 
 The scripts in this repository are provided in the hope that they are useful,

bin/__tests__/react-codemod-test.js

@@ -98,6 +98,7 @@ describe('runTransform', () => {
       transformer: 'rename-unsafe-xyz'
     });
     expect(console.log).toBeCalledWith(
+      // eslint-disable-next-line max-len
       `Executing command: jscodeshift --verbose=2 --ignore-pattern=**/node_modules/** --parser flow --transform ${path.join(
         transformerDirectory,
         'rename-unsafe-xyz.js'
@@ -115,6 +116,7 @@ describe('runTransform', () => {
       transformer: 'rename-unsafe-lifecycles'
     });
     expect(console.log).toBeCalledWith(
+      // eslint-disable-next-line max-len
       `Executing command: jscodeshift --dry --verbose=2 --ignore-pattern=**/node_modules/** --parser flow --transform ${path.join(
         transformerDirectory,
         'rename-unsafe-lifecycles.js'
@@ -132,6 +134,7 @@ describe('runTransform', () => {
       transformer: 'rename-unsafe-lifecycles'
     });
     expect(console.log).toBeCalledWith(
+      // eslint-disable-next-line max-len
       `Executing command: jscodeshift --dry --verbose=2 --ignore-pattern=**/node_modules/** --parser tsx --extensions=tsx,ts --transform ${path.join(
         transformerDirectory,
         'rename-unsafe-lifecycles.js'
@@ -152,6 +155,7 @@ describe('runTransform', () => {
       transformer: 'rename-unsafe-lifecycles'
     });
     expect(console.log).toBeCalledWith(
+      // eslint-disable-next-line max-len
       `Executing command: jscodeshift --dry --verbose=2 --ignore-pattern=**/node_modules/** --parser babel --transform ${path.join(
         transformerDirectory,
         'rename-unsafe-lifecycles.js'

bin/cli.js

@@ -27,7 +27,7 @@ function checkGitStatus(force) {
     if (force) {
       console.log(`WARNING: ${errorMessage}. Forcibly continuing.`);
     } else {
-      console.log('Thank you for using jest-codemods!');
+      console.log('Thank you for using react-codemods!');
       console.log(
         chalk.yellow(
           '\nBut before we continue, please stash or commit your git changes.'
@@ -46,7 +46,7 @@ function runTransform({ files, flags, parser, transformer, answers }) {
 
   let args = [];
 
-  const { dry, print } = flags;
+  const { dry, print, explicitRequire } = flags;
 
   if (dry) {
     args.push('--dry');
@@ -55,6 +55,10 @@ function runTransform({ files, flags, parser, transformer, answers }) {
     args.push('--print');
   }
 
+  if (explicitRequire === 'false') {
+    args.push('--explicit-require=false');
+  }
+
   args.push('--verbose=2');
 
   args.push('--ignore-pattern=**/node_modules/**');
@@ -209,7 +213,7 @@ function run() {
     `
     },
     {
-      boolean: ['force', 'dry', 'print', 'help'],
+      boolean: ['force', 'dry', 'print', 'explicit-require', 'help'],
       string: ['_'],
       alias: {
         h: 'help'