-
Notifications
You must be signed in to change notification settings - Fork 7.3k
add description default private for repo create #891
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
Conversation
| repoCreateCmd.Flags().Bool("enable-issues", true, "Enable issues in the new repository") | ||
| repoCreateCmd.Flags().Bool("enable-wiki", true, "Enable wiki in the new repository") | ||
| repoCreateCmd.Flags().Bool("public", false, "Make the new repository public") | ||
| repoCreateCmd.Flags().Bool("public", false, "Make the new repository public (default: private)") |
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.
This makes sense, but since it's a boolean flag I think a more accurate help string is (default: false). do you have an opinion, @ampinsk ?
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.
@vilmibm, if we will think about create and use the repository I guess that default: false is a good way because when we create the repository on Github.com the public option is the default. I didn't remember that.
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.
I don't feel strongly either way but I have to think less when it says default: private. Totally hear that default: false fits better with our system though!
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.
@vilmibm sorry, I understand now what do you want to say with (default: false), ignore my first reply. I totally agree with @ampinsk, I have to think less with (default: private), however, as it's a boolean flag the value (default: false) is a good way. Can I close this pull request and open another with this change?
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.
default: false would be more "correct", but I agree with @ampinsk that default: private is better since you have you think less.
| repoCreateCmd.Flags().Bool("enable-issues", true, "Enable issues in the new repository") | ||
| repoCreateCmd.Flags().Bool("enable-wiki", true, "Enable wiki in the new repository") | ||
| repoCreateCmd.Flags().Bool("public", false, "Make the new repository public") | ||
| repoCreateCmd.Flags().Bool("public", false, "Make the new repository public (default: private)") |
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.
default: false would be more "correct", but I agree with @ampinsk that default: private is better since you have you think less.
|
Duplicate of # |
I suggest using simple information on help command like (default: private) that it can be clean for users to understand that the repository will be created as private.
Fixes #890