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

Async package #625

Merged
merged 51 commits into from Mar 3, 2019

Conversation

Projects
None yet
1 participant
@codecadwallader
Copy link
Owner

codecadwallader commented Mar 3, 2019

No description provided.

codecadwallader added some commits Feb 16, 2019

Make the SettingsMonitor async, requiring a JoinableTaskFactory since…
… event handler patterns are not async friendly.
Delete all disposal logic for event handlers. Dispose and async don't…
… play well together, and looking at other examples it may not be necessary.

codecadwallader added some commits Feb 20, 2019

Downgrade ComponentModelHost to v15.8 to be in line with other depend…
…encies. Resolved a loading issue with the OutliningSynchronizationManager.
Remove a throw statement as accessing the SearchString on the main th…
…read is shown to be valid in the docs.
Change event handlers to async void (accepted convention) so that an …
…async method can be awaited. Pragma statements to turn off the warning.
Undo a lot of the ThrowIfNotOnUIThread suggestions. These interfere w…
…ith background loading which still appears to be a valid use case of the EnvDTE APIs.
Delete old unused mocks and package/tool window tests. They have been…
… disabled for awhile and getting them to work isn't worth the effort since simply starting the application proves the same thing.

@codecadwallader codecadwallader added this to the v11.0 milestone Mar 3, 2019

@codecadwallader codecadwallader merged commit 9690bb0 into master Mar 3, 2019

2 checks passed

continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@codecadwallader codecadwallader deleted the async-package branch Mar 3, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.