Skip to content

Use OWNER:BRANCH syntax for cross-repo PRs#63

Merged
mislav merged 1 commit intomasterfrom
branch-from-fork
Nov 15, 2019
Merged

Use OWNER:BRANCH syntax for cross-repo PRs#63
mislav merged 1 commit intomasterfrom
branch-from-fork

Conversation

@mislav
Copy link
Copy Markdown
Contributor

@mislav mislav commented Nov 14, 2019

This affects pr status and pr list.

This affects `pr status` and `pr list`.
Copy link
Copy Markdown
Contributor

@probablycorey probablycorey left a comment

Choose a reason for hiding this comment

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

The code looks good, I ran into one when I tried this out though. In the desktop/desktop repo look at the label for issue 8603, it has an empty repository owner. This could be because the user that opened the PR is spammy?

@mislav
Copy link
Copy Markdown
Contributor Author

mislav commented Nov 14, 2019

@probablycorey Good catch! Yes it looks like that PR is spammy.

@mislav mislav merged commit e29b375 into master Nov 15, 2019
@mislav mislav deleted the branch-from-fork branch November 15, 2019 08:28
Stonre pushed a commit to Stonre/strands-fork that referenced this pull request Jul 21, 2025
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