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

Ensure that clone works when Visual Studio is first launched #2219

Merged
merged 4 commits into from Feb 8, 2019

Conversation

@jcansdale
Copy link
Collaborator

@jcansdale jcansdale commented Feb 6, 2019

What this PR does

IGitActionsExt is proffered by SccProviderPackage, but isn't advertised. To ensure that getting IGitActionsExt doesn't return null, we first request the IGitExt which is advertised. This forces SccProviderPackage to load and proffer IGitActionsExt.

How to test

Due to issues installing MEF components when testing using F5, I'd recomend installing this VSIX using DogfoodVsix.

  1. Open Visual Studio 2019 Preview 3
  2. Select Tools > Options... and set Current source control plug-in to None.
    image
  3. Close and relaunch Visual Studio
  4. Clone or check out code > GitHub
  5. Select and Clone a repository
  6. The repository should clone

To do

Unfortunately Team Explorer / GitHub for Visual Studio doesn't notice that a GitHub repository has been loaded and the GitHub section doesn't appear on the Team Explorer - Home page.

image

At least the clone works and the banner will appear next time the solution is loaded or the current branch changes. I don't think this is a ship blocker. I'll work on a fix in another PR.

Fixes #2218

IGitActionsExt is proffered by SccProviderPackage, but isn't
advertised. To ensure that getting IGitActionsExt doesn't return null,
we first request the SccService which is advertised. This forces
SccProviderPackage to load and proffer IGitActionsExt.
jcansdale added 2 commits Feb 7, 2019
Force SccProviderPackage to load by getting IGitExt instead of the
internal SccService.
@jcansdale jcansdale merged commit 27e599f into master Feb 8, 2019
3 checks passed
3 checks passed
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
github.VisualStudio Build #20190208.21 succeeded
Details
@jcansdale jcansdale deleted the fixes/2218-clone-on-first-launch branch Feb 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.