Skip to content

Use PythonInterpreterLocatorService and other classes only when in experiment#15152

Merged
karrtikr merged 14 commits into
microsoft:mainfrom
karrtikr:pythonlocator
Jan 15, 2021
Merged

Use PythonInterpreterLocatorService and other classes only when in experiment#15152
karrtikr merged 14 commits into
microsoft:mainfrom
karrtikr:pythonlocator

Conversation

@karrtikr
Copy link
Copy Markdown

@karrtikr karrtikr commented Jan 14, 2021

Because registerLegacyDiscoveryForIOC is now an async function, I had to propogate async all the way up everywhere.

@karrtikr karrtikr added the no-changelog No news entry required label Jan 14, 2021
@karrtikr karrtikr requested a review from karthiknadig January 14, 2021 23:17
@karrtikr karrtikr marked this pull request as ready for review January 15, 2021 01:00
@github-actions github-actions Bot requested a review from int19h January 15, 2021 01:01
@karrtikr karrtikr changed the title Only use PythonInterpreterLocatorService when in experiment Register and use PythonInterpreterLocatorService only when in experiment Jan 15, 2021
@codecov-io
Copy link
Copy Markdown

codecov-io commented Jan 15, 2021

Codecov Report

Merging #15152 (e8f5b9b) into main (db6cd4b) will decrease coverage by 0%.
The diff coverage is 60%.

@@          Coverage Diff           @@
##            main   #15152   +/-   ##
======================================
- Coverage     65%      65%   -1%     
======================================
  Files        561      561           
  Lines      26525    26540   +15     
  Branches    3803     3806    +3     
======================================
+ Hits       17271    17277    +6     
- Misses      8552     8560    +8     
- Partials     702      703    +1     
Impacted Files Coverage Δ
src/client/interpreter/contracts.ts 100% <ø> (ø)
src/client/interpreter/interpreterService.ts 62% <20%> (-2%) ⬇️
src/client/pythonEnvironments/legacyIOC.ts 37% <80%> (+1%) ⬆️

@karrtikr karrtikr changed the title Register and use PythonInterpreterLocatorService only when in experiment Use PythonInterpreterLocatorService and other classes only when in experiment Jan 15, 2021
Comment thread src/client/pythonEnvironments/legacyIOC.ts Outdated
Comment thread src/client/interpreter/interpreterService.ts Outdated
Comment thread src/client/interpreter/interpreterService.ts Outdated
@karrtikr karrtikr requested a review from karthiknadig January 15, 2021 19:33
@karrtikr karrtikr merged commit b155be4 into microsoft:main Jan 15, 2021
@karrtikr karrtikr deleted the pythonlocator branch January 15, 2021 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants