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'