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.

Sign up for GitHub

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

Jump to bottom

CLI - fix basic errors #112

Closed
3 of 19 tasks
ggendre opened this issue May 12, 2016 · 2 comments
Closed
3 of 19 tasks

CLI - fix basic errors #112

ggendre opened this issue May 12, 2016 · 2 comments
Assignees
@ggendre
Labels
enhancement in progress
Projects
Cobalt 0.6

Comments

@ggendre
Copy link
Contributor

ggendre commented May 12, 2016 •
edited

Exemple of errors that could be avoided

  • avoid user to enter spaces in user project name
  • Gentle warning if sample name does not match any known repository
  • Fix error when saying no to "erase?"
  • Upgrade all node dependencies
  • spaces in current paths on Windows (maybe linux, not mac)
  • Remove .gitfolder from cloned repo

Remove nodegit in favor of something better

  • Commit on CLI branche issue-1.
  • Gentle warning if git is not installed
  • try on macOS with git not installed
  • try on windows with git not installed
  • try on linux with git not installed

Dont forget to check and double-check :

  • try all samples on macOS with node 6 stable
  • try all samples on macOS with node 4 LTS
  • try all samples on windows with node 6 stable
  • try all samples on windows with node 4 LTS
  • try all samples on linux with node 4 LTS
  • try all samples on linux with node 6 stable

And finally

  • update doc to install git in your machine's path first
  • write a blog post
@ggendre ggendre modified the milestone: 0.6 May 26, 2016
@ggendre
Copy link
Contributor Author

ggendre commented Jun 8, 2016

added task to update nodegit to latest as discussed here : cobaltians/Cobaltians-CLI#1

@ggendre ggendre added the ready label Jun 9, 2016
@ggendre ggendre self-assigned this Jul 15, 2016
@ggendre ggendre added in progress and removed ready labels Jul 15, 2016
ggendre added a commit to cobaltians/Cobaltians-CLI that referenced this issue Jul 19, 2016
@ggendre
cobaltians/Cobalt#112 Cli asks for folder erase. Regex for app name. …
190cd09 
…template name check in known templates
@ggendre
Copy link
Contributor Author

ggendre commented Jul 19, 2016

started to work on cli enhancement on branch issue-1 of cobaltians-cli.

ggendre added a commit to cobaltians/Cobaltians-CLI that referenced this issue Jul 19, 2016
@ggendre
cobaltians/Cobalt#112 reject if no platform
c75df39
@ggendre ggendre removed this from the 0.6 milestone Aug 23, 2017
@ggendre ggendre closed this as completed Feb 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ggendre ggendre

Labels
enhancement in progress
Projects
No open projects
Cobalt 0.6
In Progress
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ggendre