-
Notifications
You must be signed in to change notification settings - Fork 27
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
Add support for pydevd --continue
#74
Add support for pydevd --continue
#74
Conversation
if pc.env["WRAPPER_VERBOSE"] != "" { | ||
cmdline = append(cmdline, "--DEBUG") | ||
} | ||
if pc.debugMode == ModePydevdPycharm { | ||
// From the pydevd source, PyCharm wants multiproc | ||
cmdline = append(cmdline, "--multiproc") | ||
} | ||
if !pc.wait { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for my understanding - how will this get passed in? does skaffold hardcode this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is from the --wait
flag. It's currently not supported in Skaffold (GoogleContainerTools/skaffold#4870).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change LGTM, there was just one other comment in here mainly just for my understanding
This applies the change in fabioz/PyDev.Debugger#199 to pydevd, and adds support to the Python launcher to use this new
--continue
flag.