Skip to content

More clear error message for invalid debug configuration - #19436

Merged
Paula (paulacamargo25) merged 2 commits into
microsoft:mainfrom
scottshambaugh:debug_tests_error_message
Jul 21, 2022
Merged

More clear error message for invalid debug configuration#19436
Paula (paulacamargo25) merged 2 commits into
microsoft:mainfrom
scottshambaugh:debug_tests_error_message

Conversation

@scottshambaugh

@scottshambaugh Scott Shambaugh (scottshambaugh) commented Jul 9, 2022

Copy link
Copy Markdown

This closes #18963, and actually resolves #17150, by updating an error message string.

@karthiknadig Karthik Nadig (karthiknadig) added bug Issue identified by VS Code Team member as probable bug skip tests Updates to tests unnecessary labels Jul 14, 2022
@karthiknadig

Karthik Nadig (karthiknadig) commented Jul 14, 2022

Copy link
Copy Markdown
Member

Scott Shambaugh (@scottshambaugh) The changes look good, only thing is you might have to run prettier on the tsfile:

src/client/debugger/extension/configuration/debugConfigurationService.ts
Code style issues found in the above file(s). Forgot to run Prettier?

@scottshambaugh

Scott Shambaugh (scottshambaugh) commented Jul 14, 2022

Copy link
Copy Markdown
Author

Thanks Karthik Nadig (@karthiknadig), I'm having issues running prettifier locally but this checks out in the playground now. Not sure how to rerun the workflow to get it to check it.

@karthiknadig

Copy link
Copy Markdown
Member

Scott Shambaugh (@scottshambaugh) I pushed a linting fix to your branch. It should pass now.

@scottshambaugh

Copy link
Copy Markdown
Author

Karthik Nadig (@karthiknadig) thank you!

@paulacamargo25
Paula (paulacamargo25) merged commit b090272 into microsoft:main Jul 21, 2022
Wes McKinney (wesm) pushed a commit to posit-dev/positron that referenced this pull request Mar 28, 2024
…scode-python#19436)

* More clear error message for invalid debug configuration

* Address linting issues.

Co-authored-by: Scott Shambaugh <scottshambaugh@users.noreply.github.com>
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Issue identified by VS Code Team member as probable bug skip tests Updates to tests unnecessary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make configuration error message when debugging Python tests more helpful Error message when debugging Python tests is unhelpful

3 participants