Skip to content

Commit

Permalink
Suggest "yarn build" rather than "yarn run build" (facebook#1800)
Browse files Browse the repository at this point in the history
* Fix for issue facebook#1798: Suggested 'yarn build' versus 'yarn run build'

* remove 'run' from 'yarn test' command as well

* conditionally show 'run' if Yarn is not available
  • Loading branch information
geoffdavis92 authored and gaearon committed Mar 12, 2017
1 parent 33721da commit c40d031
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions scripts/init.js
Original file line number Diff line number Diff line change
Expand Up @@ -149,13 +149,17 @@ module.exports = function(
console.log(chalk.cyan(` ${displayedCommand} start`));
console.log(' Starts the development server.');
console.log();
console.log(chalk.cyan(` ${displayedCommand} run build`));
console.log(
chalk.cyan(` ${displayedCommand} ${useYarn ? '' : 'run '}build`)
);
console.log(' Bundles the app into static files for production.');
console.log();
console.log(chalk.cyan(` ${displayedCommand} test`));
console.log(' Starts the test runner.');
console.log();
console.log(chalk.cyan(` ${displayedCommand} run eject`));
console.log(
chalk.cyan(` ${displayedCommand} ${useYarn ? '' : 'run '}eject`)
);
console.log(
' Removes this tool and copies build dependencies, configuration files'
);
Expand Down

0 comments on commit c40d031

Please sign in to comment.