-
Notifications
You must be signed in to change notification settings - Fork 265
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
move deploy
and login
to spin cloud
#1139
Conversation
6f99e70
to
fd67c3d
Compare
109948d
to
394049b
Compare
Signed-off-by: Matthew Fisher <matt.fisher@fermyon.com>
394049b
to
8ac7598
Compare
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.
The command help for spin deploy
and spin cloud deploy
look different. I think we should make these the same?
$ spin deploy --help
spin-deploy
Deploy a Spin application
USAGE:
spin deploy [OPTIONS]
$ spin cloud deploy --help
spin-cloud-deploy
Package and upload an application to the Fermyon Platform
USAGE:
spin cloud deploy [OPTIONS]
Signed-off-by: Matthew Fisher <matt.fisher@fermyon.com>
Nice catch. This seems to be due to the fact that Fixed in 7b08785. |
Signed-off-by: Matthew Fisher <matt.fisher@fermyon.com>
Did the same in 4e62660 for |
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.
Thanks for the changes. This looks great 👍
closes #1117