-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pr-branches
doesn't show the branches when they're from a fork
#2943
Comments
It only shows if the branch is a local branch |
Well, yes, that's the bug I'm reporting here, I guess I should have made the title more clear. I'm rather sure that is not intended behaviour, as there's an |
pr-branches
doesn't show the branches for my PRs?pr-branches
doesn't show the branches when they're from a fork
The exact situations in which to display the branch name was discussed in the original PR. This was done because often this piece of information isn’t useful, e.g. I don’t care what branch I’m guessing in this case that again it’s due to the case, rather than our logic. |
@fregante sorry to bother, but I don't quite get what you meant to say in this sentence? Maybe it's just late but figured I'll ask. As for the issue, I don't quite get what's the use case of "From branch-name" that shows for source repo that isn't common with use cases of showing user's own branch names from a fork. You're right that user doesn't care what branch My use case is mostly the same as #190 which the original PR was supposed to solve too. |
As your previous issues, they're because your organization and repo are not in lower case, so we have troubles matching it. |
I checked, this is working as intended, it's in the logic:
As shown by: We do have some code that appears to support cross-repo branches, but it was probably left behind. Honestly it's not a feature I find particularly useful in the first place, it's mostly just clutter to me, except the Why do you need this information in the list? |
I would love for to work on forks |
My use cases involve branch management of both local repo and remote repo:
As I mentioned earlier though, for my use cases #2914 would work quite great too, but I don't know if it's something that could be added to refined-github. |
Both of those would be better solved by auto-deletion, which Refined GitHub already provides (and GitHub too, if you set it in your fork’s settings) As for the local branches, I usually just delete everything locally and pull again. Your current situation requires a lot of manual work that you could skip altogether with these 2 suggestions. |
If you mean the "Automatically delete head branches" in repo's settings, it does not work if the PR is not done in same repository |
The reasons I listed here before are solved, however I sometimes (not sure if it's that often but I once wanted to comment about it before) do wish to see the branch name on open PRs list (though I think more often I wish it for milestone view, I would need to make a separate feature request for that, if this issue gets resolved). The feature seems completely useless (and I don't mean to offend anyone here by this statement, the bolded part is the key here) to me in workflow I use where the people from the organisation make the branches on their forks and create PRs from those, instead of creating the branches on "source" repo. I only wish to see my branches, not everyone in the organisation but this right now is not fulfilled by this feature. I have asked for several things that improve that workflow: #2810, #2812, #2815, #2914 and I've been happy so far with all the things you guys added for this :) I understand if you don't want to pursue this one, though I must say that I think that what I'm requesting here would have same purpose as showing local branches so I don't really agree that this shouldn't be part of |
I'm sorry that I don't have anything more specific, the branches just don't show on the pull requests list.
pr-branches
is showing in the console but none of the PRs have info about their head branch below them and there's 20 PRs on that page and all of them are mine.URL: https://github.com/Cog-Creators/Red-DiscordBot/pulls?q=is%3Apr+sort%3Aupdated-desc+author%3Ajack1142+is%3Aopen
Screenshot:
Maybe this will help a little - on refined-github's closed PRs list, the branch is showing for PRs made from the
sindresorhus/refined-github
repo, but it doesn't show for my PRs (made from the fork):The text was updated successfully, but these errors were encountered: