Skip to content
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

vscode extension: use new debug terminal provider #15801

Merged
merged 5 commits into from
Dec 17, 2024

Conversation

RiskyMH
Copy link
Member

@RiskyMH RiskyMH commented Dec 17, 2024

fixes #15223 by implementing #15223 (comment)

Keeps the previous version existent if older vscode (which can be removed if/when we increase the min requirements of vscode version in package.json).

I have tried this with many bun versions (inc #15788) and node versions and it works well.


would appreciate others testing this too as it's a decent change

and @connor4312 was this how you were imagining it being used?

keep older version of debugger if older vscode
@RiskyMH RiskyMH requested a review from Electroid December 17, 2024 00:20
@robobun
Copy link

robobun commented Dec 17, 2024

Updated 5:01 PM PT - Dec 16th, 2024

@RiskyMH, your commit 84369a1 has some failures in #8069


🧪   try this PR locally:

bunx bun-pr 15801

@connor4312
Copy link

lgtm from the js-debug side :)

@Jarred-Sumner Jarred-Sumner merged commit f2d955f into main Dec 17, 2024
4 of 9 checks passed
@Jarred-Sumner Jarred-Sumner deleted the riskymh/vscode-debug-terminal branch December 17, 2024 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bun vscode extension breaks "javascript debug terminal" for node
4 participants