Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Make VSGitExt use GitSccProvider UIContext - #1926

Merged
grokys merged 2 commits into
masterfrom
fixes/1924-use-GitSccProvider-UIContext
Sep 20, 2018
Merged

grokys merged 2 commits into
masterfrom
fixes/1924-use-GitSccProvider-UIContext

Conversation

@jcansdale

@jcansdale jcansdale commented Sep 18, 2018 •

Copy link
Copy Markdown
Collaborator

What this PR does

It appears that using UICONTEXT.RepositoryOpen in VSGitExt doesn't work consistently.

  • Revert back to using the GitSccProvider UIContext.

Fixes #1924

It appears that using UICONTEXT.RepositoryOpen in VSGitExt doesn't work
consistently. Revert back to using the GitSccProvider UIContext.
@jcansdale jcansdale changed the title [wip] Make VSGitExt use GitSccProvider UIContext Make VSGitExt use GitSccProvider UIContext Sep 19, 2018

@grokys grokys left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd actually noticed this but assumed it was just a delay, because it did seem to load at some point. However, there's definitely no delay with this, initializes much faster 👍

@grokys
grokys merged commit 3f90793 into master Sep 20, 2018
@grokys
grokys deleted the fixes/1924-use-GitSccProvider-UIContext branch September 20, 2018 08:26
@meaghanlewis

Copy link
Copy Markdown
Contributor

Thanks for doing this @jcansdale! I'm late, but LGTM. ✨

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants