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

port debugging work (#6505) #6558

Merged
merged 1 commit into from
Jul 7, 2021
Merged

Conversation

DavidKutu
Copy link

For #6467
For #6391 its the following:

don't make up a random session ids
delete temp files (the kernel is not doing this)
use IDisposableRegistry
check if we need the vscode.debug.breakpoints on activate

  • 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!).
  • Appropriate comments and documentation strings in the code.
  • Has sufficient logging.
  • Has telemetry for enhancements.
  • Unit tests & system/integration tests are added/updated.
  • Test plan is updated as appropriate.
  • package-lock.json has been regenerated by running npm install (if dependencies have changed).

* don't fetch breakpoints on activate

* implement IDisposable on the debugging manager

* pass on session id's

* delete temp files

* lint

* add try catch

* store sessionId

* pass jupyter session id
@DavidKutu DavidKutu requested a review from a team as a code owner July 6, 2021 23:52
@DavidKutu DavidKutu changed the title debugging work (#6505) port debugging work (#6505) Jul 6, 2021
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (release-2021.07@0acfc63). Click here to learn what that means.
The diff coverage is n/a.

@@                Coverage Diff                @@
##             release-2021.07   #6558   +/-   ##
=================================================
  Coverage                   ?     69%           
=================================================
  Files                      ?     410           
  Lines                      ?   28255           
  Branches                   ?    4186           
=================================================
  Hits                       ?   19616           
  Misses                     ?    6976           
  Partials                   ?    1663           

@DavidKutu DavidKutu merged commit 1f7ae36 into release-2021.07 Jul 7, 2021
@DavidKutu DavidKutu deleted the david/portDebuggingWork branch July 7, 2021 22:16
DavidKutu pushed a commit that referenced this pull request Jul 8, 2021
* prepare release (#6403)

* Port jupyter kernel fix to release branch (#6486)

* Fix some problems running kernels with 'jupyter' or no raw kernel (#6464)

* Fix for 6409

* Jupyter server specific case for kernel data not being used

* Update changelog

* Fix interactive window debug cell, codelens tracking, go to cell, expand/collapse all from toolbar when focus is not on notebook editor (#6469) (#6493)

* fix #6423

* Debug cell fixes

* update execution count.

* Expand/collapse cell should not rely on `activeNotebookEditor`

* Format

* Refactor add/debug code codepaths

* Prettier

Co-authored-by: rebornix <penn.lv@gmail.com>

Co-authored-by: rebornix <penn.lv@gmail.com>

* enble the debugging setting (#6492)

* Port #6503 and #6504 to release (#6507)

* Don't pass registrationData twice when registering notebook serializer (#6503)

* If in Python daily insiders and VS Code Insiders opt into native interactive window (#6504)

* If in Python daily insiders and VS Code Insiders opt into native interactive window

* Oops

* Remove unused import

* Changelog

* Format

* debugging work (#6505) (#6558)

* don't fetch breakpoints on activate

* implement IDisposable on the debugging manager

* pass on session id's

* delete temp files

* lint

* add try catch

* store sessionId

* pass jupyter session id

* Remove start page in favor of a message (#6576)

* prepare release (#6590)

* prepare

* use same version in package.jpsn and changelog

* leave version at 2021.8.0

* publish release

* update package-lock

Co-authored-by: Rich Chiodo <rchiodo@users.noreply.github.com>
Co-authored-by: Joyce Er <joyceerhuiling@gmail.com>
Co-authored-by: rebornix <penn.lv@gmail.com>
Co-authored-by: Don Jayamanne <don.jayamanne@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants