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

Convert to AsyncPackage with background load #4875

Closed
madskristensen opened this issue Apr 23, 2018 · 3 comments
Closed

Convert to AsyncPackage with background load #4875

madskristensen opened this issue Apr 23, 2018 · 3 comments

Comments

@madskristensen
Copy link

For performance reasons all extensions targeting VS2015+ should convert the Package class to AsyncPackage. See more here https://docs.microsoft.com/en-us/visualstudio/extensibility/how-to-use-asyncpackage-to-load-vspackages-in-the-background

It's as easy as doing exactly what your current Package.Initialize method does, but in the Package.InitializeAsync method, make the first line await JoinableTaskFactory.SwitchToMainThreadAsync();

@RussKie
Copy link
Member

RussKie commented Apr 25, 2018

@sharwell is this something you have experience with?

@sharwell
Copy link
Member

sharwell commented Apr 25, 2018

Yes, but I'm waiting for #4747 and #4799 to be completed before starting new work on the VS side of things.

@sharwell
Copy link
Member

Also a decision needs to be made on #4752 (comment), since that could make this issue obsolete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants