Skip to content

Fix the path where we look for LS version for telemetry - #15749

Merged
Karthik Nadig (karthiknadig) merged 2 commits into
microsoft:mainfrom
karthiknadig:jedi-version-fix
Mar 22, 2021
Merged

Fix the path where we look for LS version for telemetry#15749
Karthik Nadig (karthiknadig) merged 2 commits into
microsoft:mainfrom
karthiknadig:jedi-version-fix

Conversation

@karthiknadig

Copy link
Copy Markdown
Member

Closes #15735
Closes #15724
Closes #15731

Comment thread src/client/activation/jedi/manager.ts
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #15749 (4d34636) into main (ff2bc45) will not change coverage.
The diff coverage is 0%.

❗ Current head 4d34636 differs from pull request most recent head a953ac3. Consider uploading reports for the commit a953ac3 to get more accurate results

@@          Coverage Diff           @@
##            main   #15749   +/-   ##
======================================
  Coverage     65%      65%           
======================================
  Files        551      551           
  Lines      26348    26348           
  Branches    3804     3804           
======================================
  Hits       17236    17236           
  Misses      8382     8382           
  Partials     730      730           
Impacted Files Coverage Δ
src/client/activation/jedi/manager.ts 29% <0%> (ø)

@karthiknadig
Karthik Nadig (karthiknadig) merged commit 3db0b24 into microsoft:main Mar 22, 2021
Karthik Nadig (karthiknadig) added a commit to karthiknadig/vscode-python that referenced this pull request Mar 22, 2021
)

* Fix the path where we look for LS version for telemetry

* Add tracing when version retreival fails
Karthik Nadig (karthiknadig) added a commit that referenced this pull request Mar 23, 2021
* Fix the path where we look for LS version for telemetry

* Add tracing when version retreival fails
@karthiknadig
Karthik Nadig (karthiknadig) deleted the jedi-version-fix branch March 24, 2021 00:39
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 skip tests Updates to tests unnecessary

Projects

None yet

5 participants