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

Fix #130: create-site can use GitHub templates #271

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lubien
Copy link
Contributor

@lubien lubien commented Jun 18, 2019

  • Use yargs for a more powerful arg parsing experience.
  • Use download-git-repo as it was @egoist choice on SAO
    lib for the same purpose.
  • Download repos via HTTPS.

Summary

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Docs
  • Build-related changes
  • Other, please describe:

If changing the UI/CSS related code, please provide the before/after screenshot:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

  • Chrome
  • Firefox
  • Safari
  • Edge
  • IE

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature
  • Related documents have been updated
  • Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

You might want a template to test, use lubien/saber-template-rin

@netlify
Copy link

netlify bot commented Jun 18, 2019

Deploy preview for saber ready!

Built with commit fb2fd48

https://deploy-preview-271--saber.netlify.com

@krmax44
Copy link
Contributor

krmax44 commented Jun 18, 2019

Just wanted to note that all Saber projects use yarn, so the package-lock.json should probably be a yarn.lock.

@lubien
Copy link
Contributor Author

lubien commented Jun 18, 2019

@krmax44 my bad. I went with the habit of npm installing without checking for yarn even tho I've seen it's lock in the project a lot 😖

Will fix (even tho this issue is on halt, just so I don't forget)

* Use yargs for a more powerful arg parsing experience.
* Use download-git-repo as it was @egoist choice on SAO
lib for the same purpose.
* Download repos via HTTPS.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants