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

Improve resiliency when invoking DTE #20

Merged
merged 1 commit into from
Feb 12, 2021
Merged

Improve resiliency when invoking DTE #20

merged 1 commit into from
Feb 12, 2021

Conversation

kzu
Copy link
Member

@kzu kzu commented Feb 12, 2021

Various minor improvements including:

  • Not starting the active document monitoring until initial build completes
  • Retrying opening of the startup file if ExecuteCommand fails (IDE might be busy doing something else on the UI thread)
  • Not trying to open startup file if we can't determine which one
  • Some troubleshooting aids by capturing thrown exceptions for Debug.Fail diagnostics

Fixes #18

Various minor improvements including:

* Not starting the active document monitoring until initial build completes
* Retrying opening of the startup file if ExecuteCommand fails (IDE might be busy doing something else on the UI thread)
* Not trying to open startup file if we can't determine which one
* Some troubleshooting aids by capturing thrown exceptions for Debug.Fail diagnostics

Fixes #18
@kzu kzu enabled auto-merge (rebase) February 12, 2021 20:49
@kzu kzu merged commit 00c37d3 into main Feb 12, 2021
@kzu kzu deleted the dev branch February 12, 2021 20:49
@devlooped devlooped locked and limited conversation to collaborators Jan 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When changing active files quickly, opening startup file may fail
1 participant