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
Validate Mac Interpreters in the background #3909
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3909 +/- ##
=======================================
- Coverage 79% 78% -<1%
=======================================
Files 408 409 +1
Lines 19049 19102 +53
Branches 3051 3057 +6
=======================================
+ Hits 14860 14897 +37
- Misses 4187 4203 +16
Partials 2 2
|
1e42598
to
c42c84c
Compare
@@ -27,6 +28,12 @@ export class ApplicationDiagnostics implements IApplicationDiagnostics { | |||
await diagnosticsService.handle(diagnostics); | |||
} | |||
})); | |||
|
|||
// Validate the Mac interperter in the background. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DonJayamanne Perhaps I'm being dense here, but you are already validating the Mac interpreter (not in the background) in the above lines 23 to 30.
In order to improve loading times of Python Extension, we should remove validating Mac interpreter in lines 23 to 30, and only validate it in the background.
For #3908
[n/a] Test plan is updated as appropriate[n/a]package-lock.json
has been regenerated by runningnpm install
(if dependencies have changed)