-
Notifications
You must be signed in to change notification settings - Fork 589
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
Creating issue failed: No matching repository (name for source repo of my fork) found for (name of source user for my fork) #1990
Comments
@Esterni I think you need to update your remotes. What is the output of |
@alexr00 since creating this issue I've moved the project into it's own repo. My apologies for not being able to provide more information or troubleshooting on this |
The issue here: I'm working on a forked repo and archived the upstream one; it seems this plugin always tries to create issues in the upstream repo and thus fails. My repos with issues are the following:
Thanks for your help! |
@sunt05 can you try adding this setting
By default, we try to be smart and look for an |
Thanks for following up this issue. I'm happy to provide more diagnostic info if needed: I really like this feature and would like to use it in a forked repo. |
I'm unable to create the PR without an upstream defined. Which seems logical, but the extension seems like it knows about the upstream without me configuring it.
The extension recognizes the "parent repo", but is unable to create the PR for that repo with a "no matching repository" found error. If you manually add the upstream to the remotes, then the PR works. Are we expecting people to add the upstream, or should GHRPI handle that logic for them if they don't have it defined? |
We're expecting people to add the upstream, but we can do a better job of nudging them to do this. If you don't set the upstream, we also won't try to fetch PRs from it in the tree. (A case where you might intentionally not set an upstream is If you've forked a project with the intent to diverge from the original.) Instead of automatically fetching data from the upstream everywhere, we could show a one-time notification if we detect something is a fork, asking if an upstream remote should be added, or show a message in the create PR view about it |
Yeah - I like the upstream notification. Perhaps we could add it when they try to initiate the PR. This will match what people who are used to GitHub expect. |
Added feature request to automatically create the upstream if it doesn't exist. |
@sunt05 we have added better upstream support. If you are still seeing this issue then please comment here and I'll reopen this issue and investigate further. |
@sunt05 thanks for commenting back so quickly. We tried to do something smart when a repo is a fork, but we should not do that if you have your You can try out the fix in our nightly build (extension id |
Great, thanks @alexr00 ! Will try this tomorrow. |
Confirmed: it's working now with the nightly build! Many thanks @alexr00! |
Issue Type: Bug
Currently working on a fork that has since been renamed. Attempting to create an issue using any method (# TODO, Create Issue..., Create issue from selection...) gives a notification that no matching repos were found, but the names given are for the original repo and author names.
Unsure of the steps to reproduce specifically. I'm fairly new to using git overall, but have had guidance from someone more experienced. I'm told there's nothing particularly special about how I'm working with this.
Extension version: 0.17.0
VS Code version: Code 1.47.0 (d5e9aa0227e057a60c82568bf31c04730dc15dcd, 2020-07-09T08:02:06.629Z)
OS version: Windows_NT x64 10.0.18362
System Info
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
oop_rasterization: disabled_off
protected_video_decode: unavailable_off
rasterization: enabled
skia_renderer: disabled_off_ok
video_decode: enabled
viz_display_compositor: enabled_on
viz_hit_test_surface_layer: disabled_off_ok
webgl: enabled
webgl2: enabled
The text was updated successfully, but these errors were encountered: