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

cli self update #218

Merged
merged 3 commits into from
May 7, 2016
Merged

cli self update #218

merged 3 commits into from
May 7, 2016

Conversation

tanner0101
Copy link
Member

@tanner0101 tanner0101 commented May 7, 2016

Allows the CLI to update itself.

If this is approved, I will remove the Homebrew support.

vapor self-update

Edit:

Also adds an option for OSX to generate and open Xcode projects.

vapor xcode

Fixes #217

@tanner0101 tanner0101 self-assigned this May 7, 2016
@tanner0101 tanner0101 added the enhancement New feature or request label May 7, 2016
@tanner0101 tanner0101 added this to the 0.8 milestone May 7, 2016
@tanner0101
Copy link
Member Author

@qutheory/contributors let me know if this looks good.

@p4t5h3
Copy link
Contributor

p4t5h3 commented May 7, 2016

We could avoid one redirect, if cli.qutheory.io redirects to GitHub via HTTPS directly.

Resolving cli.qutheory.io... 159.203.150.153
Connecting to cli.qutheory.io|159.203.150.153|:80... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: http://raw.githubusercontent.com/qutheory/vapor/master/vapor [following]
--2016-05-07 19:54:00--  http://raw.githubusercontent.com/qutheory/vapor/master/vapor
Resolving raw.githubusercontent.com... 185.31.17.133
Connecting to raw.githubusercontent.com|185.31.17.133|:80... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: https://raw.githubusercontent.com/qutheory/vapor/master/vapor [following]
--2016-05-07 19:54:00--  https://raw.githubusercontent.com/qutheory/vapor/master/vapor
Connecting to raw.githubusercontent.com|185.31.17.133|:443... connected.
HTTP request sent, awaiting response... 200 OK

Besides that, it looks good and works here.

@tanner0101
Copy link
Member Author

Good idea. Changed it.

@tanner0101 tanner0101 merged commit 59d985d into master May 7, 2016
@tanner0101 tanner0101 deleted the cli-self-update branch May 7, 2016 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants