Skip to content
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

Interactive setup #378

Merged
merged 5 commits into from
Feb 7, 2017
Merged

Interactive setup #378

merged 5 commits into from
Feb 7, 2017

Conversation

ccpricenytimes
Copy link
Contributor

Makes setup interactive rather than flag based

};


// Cleans and reinstalls node modules.
const installUserDependencies = () => {
logger.info('Cleaning node modules and reinstalling. This may take a couple of minutes...');
const result = shell.exec(`rm -rf ${userNodeModulesPath} && ${ypm} install`);
const result = shell.exec(`rm -rf ${paths.userNodeModulesPath} && ${ypm} install`);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shell.rm('-rf', ...)

@delambo delambo merged commit d44e561 into master Feb 7, 2017
@delambo delambo deleted the interactive-setup branch February 7, 2017 23:55
tizmagik added a commit that referenced this pull request Feb 17, 2017
* master: (26 commits)
  Expand version range of yarn for Travis (#400)
  update migration guide to 0.4.0 (#398)
  Update CHANGELOG.md
  adds 040 final version (#392)
  0.4.0 docs (#338)
  adds yarn version check to bootstrap (#391)
  Making kyt a dependency rather than a devDependency (#387)
  adds more functional tests (#388)
  rc5 (#384)
  fixes stylelintrc glob so that it respects file extensions (#383)
  prep work for rc3 (#381)
  Interactive setup (#378)
  Update webpack.base.js (#365)
  Adds monorepo bootstrap (#377)
  fixes linter peer dependencies (#380)
  Fix typo (#379)
  kyt-cli version bump (#376)
  updates the kyt-cli version (#370)
  fixes cli starter kyt path bug (#369)
  Adds prep for 0.4.0 release candidate (#368)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants