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

move deploy and login to spin cloud #1139

Merged
merged 3 commits into from Feb 15, 2023
Merged

Conversation

bacongobbler
Copy link
Member

@bacongobbler bacongobbler commented Feb 14, 2023

closes #1117

src/commands.rs Outdated Show resolved Hide resolved
src/commands/cloud.rs Outdated Show resolved Hide resolved
@bacongobbler bacongobbler force-pushed the spin-cloud branch 2 times, most recently from 6f99e70 to fd67c3d Compare February 14, 2023 18:10
@bacongobbler bacongobbler marked this pull request as ready for review February 14, 2023 18:20
src/commands/cloud.rs Outdated Show resolved Hide resolved
@bacongobbler bacongobbler force-pushed the spin-cloud branch 3 times, most recently from 109948d to 394049b Compare February 14, 2023 20:12
Signed-off-by: Matthew Fisher <matt.fisher@fermyon.com>
Copy link
Member

@michelleN michelleN left a 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>
@bacongobbler
Copy link
Member Author

bacongobbler commented Feb 15, 2023

Nice catch. This seems to be due to the fact that DeployCommand uses the clap(about = "") macro, while spin cloud deploy uses the docstring attached to the subcommand.

Fixed in 7b08785.

Signed-off-by: Matthew Fisher <matt.fisher@fermyon.com>
@bacongobbler
Copy link
Member Author

Did the same in 4e62660 for spin login.

Copy link
Member

@michelleN michelleN left a 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 👍

@bacongobbler bacongobbler merged commit ca0b046 into fermyon:main Feb 15, 2023
@bacongobbler bacongobbler deleted the spin-cloud branch February 15, 2023 20:01
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.

move cloud commands under spin cloud
7 participants