Skip to content
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

Pull request where head is deleted not listing #4613

Open
justint opened this issue Mar 11, 2023 · 4 comments
Open

Pull request where head is deleted not listing #4613

justint opened this issue Mar 11, 2023 · 4 comments
Labels
feature-request Request for new features or functionality

Comments

@justint
Copy link

justint commented Mar 11, 2023

Type: Bug

  1. Clone justint/papaya (https://github.com/justint/papaya) repository to local computer.
  2. Open the cloned repository in VSCode.
  3. Click the "GitHub" sidebar icon; note that no pull requests display in the list, even though there is a currently open pull request in that repository (#8: Add dark mode, anchor links, utterances support). Refreshing the list does not fix it.

Side note: that PR was once closed and had been re-opened by me a while ago, perhaps that has something to do with it not showing up?


Extension version: 0.60.0
VS Code version: Code 1.76.1 (Universal) (5e805b79fcb6ba4c2d23712967df89a089da575b, 2023-03-08T16:32:09.831Z)
OS version: Darwin arm64 21.6.0
Modes:
Sandboxed: No

@alexr00
Copy link
Member

alexr00 commented Mar 13, 2023

The key to why this happens is this:

image

We don't show PRs in the tree if the head was deleted. When the head is deleted we don't let you check on the PR in the extension, leaving you with minimal benefit from having it in the list.

@alexr00
Copy link
Member

alexr00 commented Mar 13, 2023

It does seem that this is confusing though. Leaving open to address in the future.

@alexr00 alexr00 added the feature-request Request for new features or functionality label Mar 13, 2023
@alexr00 alexr00 changed the title Pull request not listing Pull request where head is deleted not listing Mar 13, 2023
@kohanyirobert
Copy link

In the comment section of #4800 you've written @alexr00 that

When the head branch of the PR is deleted GitHub is missing some of the properties of the PR that we currently require in the extension. Because of this, we filter out PRs where the head was deleted

Above you wrote

When the head is deleted we don't let you check on the PR in the extension

Indeed it is confusing, could you elaborate what is missing that would be otherwise required by the extension. For future reference for other if for anything else?

For my use-case it'd be enough to see the PR's title/provide a link even if the head is deleted, I wouldn't need any other details.

@seanroelofs
Copy link

seanroelofs commented Apr 1, 2024

➕ Would love to have the ability to filter for closed PRs and view the changes they introduced in the diff editors locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality
Projects
None yet
Development

No branches or pull requests

4 participants