Skip to content

Show progress while creating a new pull request#2808

Merged
vilmibm merged 2 commits intocli:trunkfrom
KarelCoudijzer:fork-hang
Jan 21, 2021
Merged

Show progress while creating a new pull request#2808
vilmibm merged 2 commits intocli:trunkfrom
KarelCoudijzer:fork-hang

Conversation

@KarelCoudijzer
Copy link
Contributor

This PR adds progress indicators on the following places while creating a new pr.

  1. api.CreatePullRequest
  2. api.ForkRepo
  3. pushBranch

Note: I did not add any test as it is a pure visual change. If tests are necessary anyway, could someone point me out how to test such code?

closes #2716

@vilmibm vilmibm self-assigned this Jan 20, 2021
one of the three new spinners produced less than ideal output when a
push happened; the other two do enhance the create experience and i
think we can get away without one for push.
@vilmibm vilmibm merged commit d25938e into cli:trunk Jan 21, 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.

Creating fork during gh pr create hangs for several minutes

2 participants