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

Conversation

@jcansdale
Copy link
Collaborator

@jcansdale jcansdale commented Mar 2, 2018

Depends on #1396

This PR does the following:

  • Adds the custom UIContext Guids.UIContext_Git which is activated when a repository is active
  • Changes PullRequestStatusBarPackage to auto-load on UIContext_Git
  • Changes GitHubPackage to auto-load on UIContext_Git
  • Auto-load using a background thread
  • Ensure that GitHubPackage completes menu initialization
    • This is necessary when it's loaded on demand before a command executes

Fixes #1511

shana
shana previously approved these changes Mar 2, 2018
Copy link
Contributor

@shana shana left a comment

Choose a reason for hiding this comment

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

This is awesome, great job!

We no longer need to use `IVSGitExt` to initialize the PR status bar when there is an active Git repository.
We must await InitializeMenus to ensure that menus are ready.
SwitchToMainThreadAsync doesn't exit until a solution has loaded.
@jcansdale jcansdale force-pushed the fixes/1511-Git-UIContext branch from 94bce3d to d951127 Compare March 2, 2018 15:27
@jcansdale jcansdale changed the title Only auto-load GitHubPackage when there is an active repository Use custom UIContext to load GitHubPackage / PullRequestStatusBarPackage in context of active repo Mar 2, 2018
@jcansdale jcansdale changed the base branch from refactor/convert-VSGitExt-to-service to feature/show-current-pr March 2, 2018 20:08
@jcansdale jcansdale changed the base branch from feature/show-current-pr to master March 2, 2018 21:17
@jcansdale jcansdale changed the base branch from master to feature/show-current-pr March 2, 2018 21:18
@jcansdale jcansdale changed the base branch from feature/show-current-pr to master March 5, 2018 15:42
@jcansdale jcansdale merged commit 804f3b3 into master Mar 5, 2018
@jcansdale jcansdale deleted the fixes/1511-Git-UIContext branch March 5, 2018 16:29
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