Skip to content

Hide the restart session kernels from kernel picker - #9137

Merged
Don Jayamanne (DonJayamanne) merged 7 commits into
microsoft:masterfrom
DonJayamanne:hideKernels
Dec 16, 2019
Merged

Hide the restart session kernels from kernel picker#9137
Don Jayamanne (DonJayamanne) merged 7 commits into
microsoft:masterfrom
DonJayamanne:hideKernels

Conversation

@DonJayamanne

Copy link
Copy Markdown

For #8866

@codecov-io

Codecov (codecov-io) commented Dec 16, 2019

Copy link
Copy Markdown

Codecov Report

Merging #9137 into master will decrease coverage by 0.01%.
The diff coverage is 38.46%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9137      +/-   ##
==========================================
- Coverage   56.39%   56.38%   -0.02%     
==========================================
  Files         551      551              
  Lines       28840    28854      +14     
  Branches     4427     4429       +2     
==========================================
+ Hits        16264    16268       +4     
- Misses      11662    11679      +17     
+ Partials      914      907       -7
Impacted Files Coverage Δ
src/client/datascience/jupyter/jupyterSession.ts 6.52% <0%> (-0.11%) ⬇️
src/client/datascience/jupyter/jupyterExecution.ts 51.97% <0%> (-0.7%) ⬇️
...lient/datascience/jupyter/jupyterSessionManager.ts 9.63% <0%> (-0.12%) ⬇️
...atascience/jupyter/jupyterSessionManagerFactory.ts 42.85% <25%> (+1.19%) ⬆️
...ient/datascience/jupyter/kernels/kernelSelector.ts 73.83% <75%> (-1.42%) ⬇️
src/client/common/utils/version.ts 76.92% <0%> (ø) ⬆️
.../client/datascience/jupyter/jupyterConnectError.ts 50% <0%> (ø) ⬆️
src/client/datascience/cellFactory.ts 53.42% <0%> (ø) ⬆️
src/client/providers/providerUtilities.ts 86.66% <0%> (ø) ⬆️
src/client/constants.ts 75% <0%> (ø) ⬆️
... and 1 more

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 30d50bb...b69100a. Read the comment docs.

@rchiodo Rich Chiodo (rchiodo) left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

Comment thread src/client/datascience/jupyter/jupyterSession.ts Outdated

@IanMatthewHuff Ian Huff (IanMatthewHuff) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🕐

@IanMatthewHuff Ian Huff (IanMatthewHuff) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@DonJayamanne
Don Jayamanne (DonJayamanne) merged commit c4c3dd2 into microsoft:master Dec 16, 2019
Don Jayamanne (DonJayamanne) added a commit that referenced this pull request Dec 16, 2019
* Hide the restart session kernels

* Add tests

* Fix linter

* Fix tests

* Fix tests

* Address code reviews

* Fix linter
@DonJayamanne
Don Jayamanne (DonJayamanne) deleted the hideKernels branch December 17, 2019 17:28
@lock lock Bot locked as resolved and limited conversation to collaborators Dec 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants