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

Ensure arguments are generated correctly for getRemoteLauncherCommand when in debugger experiment #8712

Merged
merged 2 commits into from
Nov 21, 2019

Conversation

karthiknadig
Copy link
Member

For #8685

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Has a news entry file (remember to thank yourself!)
  • Unit tests & system/integration tests are added/updated

@codecov-io
Copy link

codecov-io commented Nov 21, 2019

Codecov Report

Merging #8712 into master will decrease coverage by <.01%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8712      +/-   ##
==========================================
- Coverage   59.17%   59.16%   -0.01%     
==========================================
  Files         521      521              
  Lines       23985    23987       +2     
  Branches     3874     3875       +1     
==========================================
  Hits        14192    14192              
- Misses       8876     8878       +2     
  Partials      917      917
Impacted Files Coverage Δ
src/client/debugger/extension/adapter/factory.ts 89.83% <33.33%> (-3.16%) ⬇️

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 416e3c4...04e5806. Read the comment docs.

@karthiknadig karthiknadig merged commit 8363ec0 into microsoft:master Nov 21, 2019
DonJayamanne pushed a commit that referenced this pull request Nov 21, 2019
…d` when in debugger experiment (#8712)

* Return new debugger arguments when in experiment

* Add news item
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
@karthiknadig karthiknadig deleted the issue8685 branch December 27, 2019 02:09
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