Skip to content

debugger: Fix a few issues with JS debugging #32918

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

Merged
merged 3 commits into from
Jun 18, 2025
Merged

Conversation

cole-miller
Copy link
Contributor

@cole-miller cole-miller commented Jun 18, 2025

  • Don't assume all located tasks come from our test runnables
  • Run tests from the right working directory
  • Scope forking behavior customization for jest and vitest more tightly, to just our test runnables
  • Standardize on $PACKAGE_MANAGER exec -- $TEST_LIBRARY ... to fix runnables not working with npm

Release Notes:

  • Debugger Beta: Fixed issues with debugging tasks from package.json and test runnables.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jun 18, 2025
@cole-miller cole-miller self-assigned this Jun 18, 2025
@cole-miller cole-miller merged commit 2f52e2d into main Jun 18, 2025
21 checks passed
@cole-miller cole-miller deleted the debugger-js-issues branch June 18, 2025 14:37
JosephTLyons pushed a commit that referenced this pull request Jun 18, 2025
- Don't assume all located tasks come from our test runnables
- Run tests from the right working directory
- Scope forking behavior customization for jest and vitest more tightly,
to just our test runnables
- Standardize on `$PACKAGE_MANAGER exec -- $TEST_LIBRARY ...` to fix
runnables not working with npm

Release Notes:

- Debugger Beta: Fixed issues with debugging tasks from package.json and
test runnables.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant