VisualStudio icon indicating copy to clipboard operation
VisualStudio copied to clipboard

Extension doesn't recognize my project as existing on GitHub

Open jaredpar opened this issue 7 years ago • 18 comments

  • GitHub Extension for Visual Studio version: 2.5.2.2566
  • Visual Studio version: 15.7.1

When I open VsVim in Visual Studio the GitHub extension window puts up the message:

This repository is not on GitHub

This is certainly not true :smile:

As an experiment I hit the "Get Started" button and attempted to publish it hoping it would get the extension to reset a bit and find the extension. The publish failed with "there is already a repository with this name" yet it still won't connect it in Visual Studio.

I looked through the extension.log file a bit and there are too many errors for me to track down this particular problem. Or give me a hint as to what is going on.

Extension log file: https://gist.github.com/jaredpar/ee2c7110c5529e9cde6e67b048d5e925

jaredpar avatar May 29 '18 21:05 jaredpar

Actually this error does look a bit interesting:

Error logging into '"https://github.com/"' as 'jaredpa' Octokit.RateLimitExceededException: API rate limit exceeded for user ID 146967.

My username is jaredpar, not jaredpa. Not sure if that's just a log trimming issue though.

jaredpar avatar May 29 '18 21:05 jaredpar

Hi @jaredpar!

Thanks for getting in touch. This is something that happens occasionally, and we've not got to the bottom of it, so I'm going to do a dump here of all the relevant information for future reference.

SQLite is failing to load the native dll, sqlite3.dll but failing. It should be looking in C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\IDE\Extensions\GitHub\GitHub\x86 but the x86 part of the path is missing from looking in your logs.

This problem has ben reported before in:

https://github.com/github/VisualStudio/issues/958 https://github.com/github/VisualStudio/issues/920

And with another extension:

https://github.com/ericsink/SQLitePCL.raw/issues/54

grokys avatar May 30 '18 08:05 grokys

The error message as it stands today doesn't include the fully probed path, so not having x86 in the log is expected.

@jaredpar We haven't been able to reproduce this problem, so hopefully you can help us finally track it down for good. I'm putting together a build with better error messages and a bit more robustness in loading sqlite, if you'd be willing to give it a try?

shana avatar May 30 '18 10:05 shana

Happy to help. Just let me know when the build is available and what other info you need. On Wed, May 30, 2018 at 3:55 AM Andreia Gaita [email protected] wrote:

The error message as it stands today doesn't include the fully probed path, so not having x86 in the log is expected.

@jaredpar https://github.com/jaredpar We haven't been able to reproduce this problem, so hopefully you can help us finally track it down for good. I'm putting together a build with better error messages and a bit more robustness in loading sqlite, if you'd be willing to give it a try?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/github/VisualStudio/issues/1709#issuecomment-393118573, or mute the thread https://github.com/notifications/unsubscribe-auth/AAI-FxhmbmJETX5rJS8jzl_ddz7W6e3Wks5t3nqZgaJpZM4USPWG .

jaredpar avatar May 30 '18 14:05 jaredpar

Any updates here? The extension is essentially unusable in it's current state.

jaredpar avatar Jun 05 '18 18:06 jaredpar

@jaredpar 👋,

I was wondering you could use procmon with filters for sqlite3.dll and devenv.exe? Hopefully that would show us where it's probing for and failing to find sqlite3.dll.

jcansdale avatar Jun 07 '18 09:06 jcansdale

I'm so sorry, it's been a rather crazy week around here and I dropped the ball on this 😢 I'll get you a build in a couple of hours!

shana avatar Jun 07 '18 10:06 shana

@jaredpar Try the v2.5.2.2803 vsix build at https://ci.appveyor.com/project/github-windows/visualstudio/build/2.5.2.2803/artifacts. It should either work or at least provide better error results.

shana avatar Jun 07 '18 14:06 shana

@shana no problem on the delay. I tried that build out and it didn't address my problem.

But I did end up finding a way to solve it. The key was adding a remote named "origin". Previously I had two remotes that looked like this:

C:\Users\jaredpar\code\VsVim> git remote -v
jaredpar        https://github.com/jaredpar/VsVim.git (fetch)
jaredpar        https://github.com/jaredpar/VsVim.git (push)
rick    https://github.com/ricksladkey/VsVim.git (fetch)
rick    https://github.com/ricksladkey/VsVim.git (push)

Once I added "origin" that just pointed to the same place as "jaredpar" then the extension started working immediately.

jaredpar avatar Jun 08 '18 16:06 jaredpar

@jaredpar,

Coincidentally I'm currently working on this part of the code-base. We assume that the GitHub repository associated with the local repository is pointed to by a remote called origin. We don't necessarily want to use the remote from the tracking branch because the user might be collaborating on a pull request originating from another user's repository.

Showing a link to and PRs from ricksladkeys repository probably wouldn't be very useful. Having said that, if there is no origin, it would probably make sense to fall back to using the remote from the tracking branch.

What do you think?

jcansdale avatar Jun 08 '18 18:06 jcansdale

@jcansdale

That makes sense to me.

jaredpar avatar Jun 09 '18 01:06 jaredpar

@jaredpar Could you upload your log with this new build? I'm still curious to see if the sqlite exceptions have stopped

shana avatar Jun 11 '18 15:06 shana

@shana

Could you upload your log with this new build? I'm still curious to see if the sqlite exceptions have stopped

Sure thing. Will try and get that to you tonight. Sorry for the delay. Been out of town for a week with limited email.

jaredpar avatar Jun 18 '18 15:06 jaredpar

@jcansdale I experienced this issue as well and spent quite some time trying to figure out what's going on (had a simple typo in my remote's name). I think it would be nice to have

  • a) a hint at the necessity to precisely name your remote "origin"
  • b) an integrated fallback

timdhoffmann avatar Oct 03 '18 10:10 timdhoffmann

@timdhoffmann 👋

I experienced this issue as well and spent quite some time trying to figure out what's going on (had a simple typo in my remote's name).

Sorry about that! I agree the current situation isn't great.

I'm thinking maybe something along the lines of this: image

I'm terrible at wording this kind of thing, so let me know if you think of anything that would be make it clearer! Clicking the title would navigate the user to an issue with more details about this.

jcansdale avatar Oct 12 '18 10:10 jcansdale

I think that experience would be perfect. The inability to specify the remote to use is a tad bit annoying. But I also realize not having origin defined is probably the 1% case. As long as there is something telling me that's the problem I can move forward.

jaredpar avatar Oct 12 '18 15:10 jaredpar

Agree, that would be helpful.

timdhoffmann avatar Oct 12 '18 15:10 timdhoffmann

Bump. My repository suddenly stopped being detected as existing on GitHub.

Xottab-DUTY avatar Apr 05 '23 12:04 Xottab-DUTY