Skip to content

Fix prompt string when creating remote repository#2962

Merged
vilmibm merged 3 commits intotrunkfrom
unknown repository
Feb 17, 2021
Merged

Fix prompt string when creating remote repository#2962
vilmibm merged 3 commits intotrunkfrom
unknown repository

Conversation

@ghost
Copy link

@ghost ghost commented Feb 13, 2021

what this changes

  • Changed prompt when you are in Git project already and use gh repo create

why

If you are in git project not pushed to remote yet,
prompt says 'This will create {reponame} in current directory. Continue?',
however, it doesn't create while it only adds remote origin.

I was going to create PR to avoid creating new directory before I knew
this behavior.
This behavior is already ideal, so I changed prompt not to scare users
like I got scared.

ss 5

ulwlu added 3 commits February 13, 2021 17:11
If you are in git project not pushed to remote yet,
prompt says 'This will create {reponame} in current directory. Continue?',
however, it doesn't create while it only adds remote origin.

I was going to create PR to avoid creating new directory before I knew
this behavior.
This behavior is already ideal, so I changed prompt not to scare users
like I got scared.
@vilmibm
Copy link
Contributor

vilmibm commented Feb 16, 2021

I think this is a helpful stopgap until we rewrite repo create as we've discussed; the wording remains confusing but is strictly less confusing with this change. Would you agree, @mislav / @samcoe ?

Copy link
Contributor

@vilmibm vilmibm left a comment

Choose a reason for hiding this comment

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

I think this change has merit (as I said in my other comment) but am waiting on merging until I hear from the other core engineers

@samcoe
Copy link
Contributor

samcoe commented Feb 17, 2021

@vilmibm I agree with you that this change makes this all less confusing and more accurate as to what will take place upon confirmation. I still don't love the verbiage, but don't have any better ideas off the top of my head.

@vilmibm vilmibm merged commit 4a897f7 into cli:trunk Feb 17, 2021
@mislav mislav mentioned this pull request Feb 17, 2021
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.

2 participants