Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@victorhsn
Copy link
Contributor

@victorhsn victorhsn commented May 8, 2020

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

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)")
Copy link
Contributor

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 ?

Copy link
Contributor Author

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.

Copy link
Contributor

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!

Copy link
Contributor Author

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?

Copy link
Contributor

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)")
Copy link
Contributor

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.

@mislav mislav merged commit e28e609 into cli:master May 13, 2020
@ghost
Copy link

ghost commented Jun 11, 2020

Duplicate of #

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.

Add default information on repo create

4 participants