Skip to content

Commit

Permalink
Fix microsoft#975: Properties parameter to the configure API has beco…
Browse files Browse the repository at this point in the history
…me required in 1.6.1
  • Loading branch information
Pavel Minaev committed Jul 7, 2022
1 parent 4f2a456 commit 74b2ab5
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/debugpy/server/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ def log_to(path):
log.log_dir = path


def configure(properties, **kwargs):
def configure(properties=None, **kwargs):
if _settrace.called:
raise RuntimeError("debug adapter is already running")

Expand Down
5 changes: 5 additions & 0 deletions tests/debugpy/test_attach.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,11 @@ def code_to_debug():
import time
from debuggee import backchannel, scratchpad

# Test different ways of calling configure().
debugpy.configure(qt="none", subProcess=True, python=sys.executable)
debugpy.configure({"qt": "none", "subProcess": True, "python": sys.executable})
debugpy.configure({"qt": "none"}, python=sys.executable)

debuggee.setup()
_, host, port, wait_for_client, is_client_connected, stop_method = sys.argv
port = int(port)
Expand Down

0 comments on commit 74b2ab5

Please sign in to comment.