Skip to content

Okay#1080

Closed
GearHead001 wants to merge 140 commits intoalias-listfrom
trunk
Closed

Okay#1080
GearHead001 wants to merge 140 commits intoalias-listfrom
trunk

Conversation

@GearHead001
Copy link
Copy Markdown

Okay gh pr checkout 99085

wingkwong and others added 30 commits February 25, 2020 01:25
Co-Authored-By: Nate Smith <[email protected]>
Co-Authored-By: Nate Smith <[email protected]>
Co-Authored-By: Nate Smith <[email protected]>
Co-Authored-By: Nate Smith <[email protected]>
Co-Authored-By: Nate Smith <[email protected]>
Our code had an unspoken assumption that only one apiClient is created
during the course of a command. Violating this assumption is fine in
almost all cases, but not when we need to do a re-auth to add a new
oauth scope to a user's token.

There is likely a more elegant solution to the problem but until then
this changes determineBaseRepo to use an existing apiClient.
mislav and others added 26 commits May 27, 2020 14:43
This was an accidental casualty of refactoring in
3aaa231
Avoids the error when `localhost` was resolved to something else other
than `127.0.0.1`:

    listen tcp 192.168.1.*:0: bind: can't assign requested address
This is available now that we've bumped githubv4
removing extra line
Done

Co-authored-by: Youssef Victor <[email protected]>
…view-pr-command

📚 docs(typo): fix typo in gh pr review command
Fix referring to loopback interface in OAuth flow
Fix showing "Authentication complete" notice after OAuth dance
@AliabbasMerchant
Copy link
Copy Markdown
Contributor

@billygriffin I think we will have to make a separate repo, just for people to test out gh 😄

@Serializm13

This comment was marked as spam.

@Serializm13

This comment was marked as spam.

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.