Skip to content

Implement global virtual environment locator#14416

Merged
karthiknadig merged 13 commits intomicrosoft:mainfrom
karthiknadig:global3
Oct 22, 2020
Merged

Implement global virtual environment locator#14416
karthiknadig merged 13 commits intomicrosoft:mainfrom
karthiknadig:global3

Conversation

@karthiknadig
Copy link
Copy Markdown
Member

@karthiknadig karthiknadig commented Oct 16, 2020

I consolidated a bunch of different virtual environment identifiers and locators. There was some clean up and code moved around. but the bulk of the changes are in GlobalVirtualEnvironment*.ts files.

@karthiknadig karthiknadig added the no-changelog No news entry required label Oct 16, 2020
@codecov-io
Copy link
Copy Markdown

codecov-io commented Oct 16, 2020

Codecov Report

Merging #14416 into main will increase coverage by 0.10%.
The diff coverage is 99.26%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #14416      +/-   ##
==========================================
+ Coverage   59.18%   59.29%   +0.10%     
==========================================
  Files         720      719       -1     
  Lines       40216    40315      +99     
  Branches     5830     5849      +19     
==========================================
+ Hits        23803    23903     +100     
  Misses      15151    15151              
+ Partials     1262     1261       -1     
Impacted Files Coverage Δ
.../locators/services/virtualEnvironmentIdentifier.ts 97.36% <97.36%> (ø)
...rc/client/pythonEnvironments/common/commonUtils.ts 100.00% <100.00%> (ø)
...pythonEnvironments/common/environmentIdentifier.ts 96.77% <100.00%> (-0.20%) ⬇️
...ocators/services/globalVirtualEnvronmentLocator.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 25fd710...88f2da5. Read the comment docs.

Comment thread src/client/pythonEnvironments/common/commonUtils.ts Outdated
Comment thread src/client/pythonEnvironments/common/commonUtils.ts Outdated
Comment thread src/client/pythonEnvironments/common/commonUtils.ts Outdated
Comment thread src/client/pythonEnvironments/common/commonUtils.ts Outdated
Comment thread src/client/pythonEnvironments/common/commonUtils.ts Outdated
@karthiknadig karthiknadig requested a review from karrtikr October 19, 2020 18:16
@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@karthiknadig karthiknadig merged commit 4333fc0 into microsoft:main Oct 22, 2020
@karthiknadig karthiknadig deleted the global3 branch October 23, 2020 05:54
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.

5 participants