Conversation
|
This is almost enough to make things work. However, we need to expose an additional port (default: ports:
- port: 8000
targetPort: http-server
name: wsgi
- port: 5678
targetPort: 5678
name: debugHowever, while the server still responded on |
|
@toastdriven you need to add the port to the resource in Tilt to forward to the debug port: |
|
I have VS Code debugging fully working as of the latest commit. However, there are a couple small things to discuss/address:
|
20304e4 to
41943a8
Compare
41943a8 to
c5a2717
Compare
c5a2717 to
9f9b478
Compare
|
We don't want to make VSCode-specific changes in the base layer since all environments will inherit them. They must either go into |
9f9b478 to
54e60a7
Compare
54e60a7 to
1b71b74
Compare
1b71b74 to
e21fc98
Compare
e21fc98 to
0f75594
Compare
| * PyCharm | ||
| * VS Code |
There was a problem hiding this comment.
Can we add a note saying any DAP-supported IDE?
| PYTHONBREAKPOINT: "" # "utils.pycharm_debugger" for pycharm, "utils.vscode_debugger" for VS Code | ||
| DEBUGGER_PORT: "" # "6400" for pycharm, "5678" for VS Code |
There was a problem hiding this comment.
Modifying it in the repo will require you to git exclude the file, which is not ideal. We can patch in these options using Tilt as well.
🎯 What needed to be done and why?
Adds support for interactive debugging in VS Code (among other editors, e.g. Neovim & emacs).
Quick summary of the issue
We have a stub in
utils/debugger.py:vscode_debugger. This fleshes that out.🆕 What is changed by this PR?
This handles a similar "import debug library & bind to port" pattern as the existing
utils.debugger.py:pycharm_debugger, but fordebugpy.This enables anything that works with DAP protocols to talk to the
backendprocess & interactively debug. This means support for:📋 Code Review Cheatsheet
What the reviewer should check