-
-
Notifications
You must be signed in to change notification settings - Fork 26.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve the output of scripts/init.js #629
Conversation
console.log(); | ||
console.log(chalk.cyan('npm run eject')); | ||
console.log(' Removes this tool and copies build dependencies, configs,') | ||
console.log(' and scripts into the app directory.') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this line and the one above it are missing semicolons in the end, can you add them back? Otherwise everything seems to be in order now 👌
Fixed 😄 |
Great, thanks for pulling it through! 👍 |
console.log(' Bundles the app into static files for production.'); | ||
console.log(); | ||
console.log(chalk.cyan('npm run eject')); | ||
console.log(' Removes this tool and copies build dependencies, configs,'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fson Can you please change "configs" to "configuration files" directly in master? I'd like to avoid jargon in messages.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
* Improve the output of scripts/init.js * Add semi-colons;
@fson Ran into major problems with git trying to merge this pull-request:
#363