Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes to starting Jupyter in a Docker container. #8715

Merged
merged 2 commits into from
Nov 21, 2019

Conversation

DonJayamanne
Copy link

For #8661

@codecov-io
Copy link

codecov-io commented Nov 21, 2019

Codecov Report

Merging #8715 into master will decrease coverage by 0.63%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8715      +/-   ##
==========================================
- Coverage   58.58%   57.95%   -0.64%     
==========================================
  Files         526      526              
  Lines       24572    24200     -372     
  Branches     3979     3974       -5     
==========================================
- Hits        14395    14024     -371     
+ Misses       9244     9241       -3     
- Partials      933      935       +2
Impacted Files Coverage Δ
src/client/datascience/jupyter/notebookStarter.ts 68.51% <100%> (+4.62%) ⬆️
...nt/datascience/jupyter/invalidNotebookFileError.ts 0% <0%> (-75%) ⬇️
.../client/datascience/jupyter/jupyterConnectError.ts 0% <0%> (-66.67%) ⬇️
...science/jupyter/jupyterKernelPromiseFailedError.ts 0% <0%> (-50%) ⬇️
...ent/datascience/jupyter/jupyterWaitForIdleError.ts 0% <0%> (-50%) ⬇️
...cience/jupyter/jupyterDebuggerNotInstalledError.ts 50% <0%> (-30%) ⬇️
src/client/language/characters.ts 12% <0%> (-21.34%) ⬇️
src/client/datascience/cellFactory.ts 51.51% <0%> (-19%) ⬇️
src/client/testing/codeLenses/main.ts 60% <0%> (-17.78%) ⬇️
src/client/constants.ts 75% <0%> (-5%) ⬇️
... and 29 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 8363ec0...2b71f28. Read the comment docs.

@DonJayamanne DonJayamanne merged commit eddbb65 into microsoft:master Nov 21, 2019
DonJayamanne added a commit that referenced this pull request Nov 21, 2019
* Ensure we generate the right args when running in docker
* Fixes to starting `Jupyter` in a `Docker` container.
karthiknadig added a commit that referenced this pull request Nov 27, 2019
* Ensure we run PR, CI pipelines on release branches (#8471) (#8474)

* Port blue icon and altair background fixes into release (#8470)

* fix blue color on export icon (#8449)

* default non-text mimetypes to white background (#8452)

* Fix cells being erased when saving and then changing focus to another… (#8482) (#8485)

* Fix cells being erased when saving and then changing focus to another cell

* Make sure to use the source even if it's empty

* Fix timeout to be what it was before. 10 seconds isn't long enough (#8504)

* Ensure wheels experiment control and experiment groups uses right wheel (#8461) (#8546)

* Ensure wheels control group uses no wheel ptvsd
* Improve condition readability in jupyter debugger

* Adjust debug adapter experiment before release  (#8540) (#8553)

* Release branch: updating change log and package json (#8570)

* Change version numbers

* Update package lock json

* Update changelog

* Fix for 7999 - CTRL+Z should not undo at the top level (#8571)

* Fix arrowing up and down to save code too (#8574)

* Prompt selecting a file to save once (#8590)

* Prompt selecting a file to save once
* Better algorithm

* After pasting code, arrow keys don't navigate in a cell. (#8587)

* Don't look for a jupyter interpreter when creating blank notebook (#8596)

* Creating a new blank notebook should not require a search for jupyter.

* Fix bug caused by other fix.

* Update change log

* Update release date in change log (#8604)

* Fix converting from a python script (#8699)

* Fix converting from a python script

* Fix linter error

* Fix problem with $$ not being put around the correct items in a LaTeX equation (#8707)

* First try

* Add news entry and fix multiples

* Add a bunch of comments

* Modify changelog and version number

* Fixes to starting Jupyter in a Docker container. (#8715)

* Ensure we generate the right args when running in docker
* Fixes to starting `Jupyter` in a `Docker` container.

* Ensure arguments are generated correctly for `getRemoteLauncherCommand` when in debugger experiment (#8712)

* Return new debugger arguments when in experiment

* Add news item

* Updates to change log

* Revert tests due to incompatibilities
@lock lock bot locked as resolved and limited conversation to collaborators Nov 28, 2019
@DonJayamanne DonJayamanne deleted the fixDockerRun branch December 17, 2019 17:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants