-
Notifications
You must be signed in to change notification settings - Fork 771
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
Removing unconventional two letter flags + adding dashes in-between two letter words. #331
Comments
Kubectl example using two-letter words with dashes:
And when referencing replication controller, it's conventionally used with a dash in-between the words:
|
Okay since #304 was merged, this can be closed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Using multi-two-letter flags is unconventional and against common CLI usages.
For example:
Should either be JUST
--rc
or--replicationcontroller
.Ideally, flags with two names should have dashes in between them, such as
--replication-controller
.Common CLI tools such as
kubectl
andoc
use these conventions and considering most of the users are coming from these tools, we should follow suit.PR is available here with these changes: #304
The text was updated successfully, but these errors were encountered: