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

Standardize CLI flags #118

Merged
merged 3 commits into from
Dec 15, 2023
Merged

Conversation

webstradev
Copy link
Contributor

I standardized all the CLI flags to kebab-case
[Closes #116 ]

Signed-off-by: Erik Westra <[email protected]>
Signed-off-by: Erik Westra <[email protected]>
@nabuskey
Copy link
Collaborator

@webstradev Thanks so much for this! Could you also update the readme file to reflect the changes you've made?

@webstradev
Copy link
Contributor Author

@webstradev Thanks so much for this! Could you also update the readme file to reflect the changes you've made?

Will do

@webstradev webstradev force-pushed the standardize-cli-flags branch from 660c0cd to 067d50e Compare December 15, 2023 18:05
@nimakaviani nimakaviani self-requested a review December 15, 2023 18:27
Copy link
Contributor

@nimakaviani nimakaviani left a comment

Choose a reason for hiding this comment

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

Thank you!

LGTM

@nabuskey nabuskey merged commit 243ba5f into cnoe-io:main Dec 15, 2023
2 checks passed
@webstradev webstradev deleted the standardize-cli-flags branch December 15, 2023 18:58
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.

Inconsistent flag name convention
3 participants