Skip to content

Conversation

@DonJayamanne
Copy link

Fixes #1986

This pull request:

  • Has a title summarizes what is changing
  • Includes a news entry file (remember to thank yourself!)
@codecov
Copy link

codecov bot commented Jun 21, 2018

Codecov Report

Merging #2032 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@ ## master #2032 +/- ## ========================================== - Coverage 75.02% 74.94% -0.09%  ========================================== Files 307 307 Lines 14060 14060 Branches 2493 2493 ========================================== - Hits 10549 10537 -12  - Misses 3374 3381 +7  - Partials 137 142 +5
Impacted Files Coverage Δ
src/client/debugger/PythonProcess.ts 45.83% <0%> (-5.42%) ⬇️
src/client/linters/lintingEngine.ts 90.26% <0%> (-0.89%) ⬇️
...rc/client/debugger/PythonProcessCallbackHandler.ts 54.27% <0%> (+0.65%) ⬆️

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 c066c32...4d5f456. Read the comment docs.

@d3r3kk
Copy link

d3r3kk commented Jun 21, 2018

@DonJayamanne I will prioritize getting the quick npm run test:unittest done today. Will likely require a bit of a hack to make the output how we want, but I'll add issues to counter that in future.

@DonJayamanne DonJayamanne merged commit 101c78d into microsoft:master Jun 21, 2018
@DonJayamanne DonJayamanne deleted the issue1986DoNotRunTestsInPreCommit branch October 2, 2018 22:45
@lock lock bot locked as resolved and limited conversation to collaborators Jul 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

3 participants