Skip to content

add description default private for repo create#891

Merged
mislav merged 1 commit into
cli:masterfrom
victorhsn:master
May 13, 2020
Merged

add description default private for repo create#891
mislav merged 1 commit into
cli:masterfrom
victorhsn:master

Conversation

@victorhsn

@victorhsn victorhsn commented May 8, 2020

Copy link
Copy Markdown
Contributor

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

Comment thread command/repo.go
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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.

Comment thread command/repo.go
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
Copy Markdown
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

ghost commented Jun 11, 2020

Copy link
Copy Markdown

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