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

Enhance some CLI Command Descriptions #14

Merged
merged 3 commits into from
Sep 23, 2024
Merged

Conversation

lborgav
Copy link
Contributor

@lborgav lborgav commented Sep 23, 2024

I’ve updated the command descriptions that were previously using Cobra’s default placeholder text.
The new descriptions provide more clarity and correctness

@pythoninthegrass pythoninthegrass mentioned this pull request Sep 23, 2024
Copy link
Owner

@MightyMoud MightyMoud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Absolute Legend! This been on my task list for a while now.
Tiny comment and LGTM.

Thanks @lborgav

cmd/deploy.go Outdated
This application is a tool to generate the needed files
to quickly create a Cobra application.`,
Short: "Deploy a new version of your application to your VPS using Sidekick",
Long: `This command deploys a new version o your application to your VPS.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small typo here of your application

Copy link
Contributor Author

@lborgav lborgav Sep 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just fixed it.
Nice and helpful project. I'd be happy to help more

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!
Happy to hear that! There will be big features we would like to build in the next couple of months. We can discuss more

@lborgav lborgav requested a review from MightyMoud September 23, 2024 15:03
@MightyMoud MightyMoud merged commit d205a85 into MightyMoud:main Sep 23, 2024
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.

2 participants