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

Skip to content

Conversation

@mislav
Copy link
Contributor

@mislav mislav commented May 7, 2020

At the time we have a reference to baseRepo, we might still not have contacted the API nor obtained any information about the default branch for the repository. This expands the PullRequests() query to always report the default branch so we may choose how to render entries that belong on the current branch.

Fixes #842 (comment) /cc @foad

At the time we have a reference to `baseRepo`, we might still not have
contacted the API nor obtained any information about the default branch
for the repository. This expands the `PullRequests()` query to always
report the default branch so we may choose how to render entries that
belong on the current branch.
@mislav mislav requested review from probablycorey and vilmibm May 7, 2020 18:56
Copy link
Contributor

@foad foad left a comment

Choose a reason for hiding this comment

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

Thank you for this 😁

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.

thank you!

@vilmibm vilmibm merged commit 77ed118 into master May 8, 2020
@mislav mislav deleted the pr-status-crash branch May 11, 2020 07:59
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.

3 participants