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

Don't prefix node-terminal with pwa #99307

Closed
roblourens opened this issue Jun 3, 2020 · 5 comments
Closed

Don't prefix node-terminal with pwa #99307

roblourens opened this issue Jun 3, 2020 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@roblourens
Copy link
Member

Verifying #95836

This would look nicer if we could use a node-terminal type instead of pwa-node-terminal, since this is a place where we actually write this into user's launch configs. Typically users could see the pwa prefix but only if they are messing around with debug types on their own. We will have to either support the pwa prefix forever or get rid of it and make some users adjust their configs at some point.

image

@connor4312
Copy link
Member

Yea. This is going to be kind of a pain in general when we swap over to replace node and chrome as well.

Maybe we just eat a breaking change then or have a quick notification with a button to remove the prefix. Removing the prefix for terminal, which is a new type, makes sense though.

@connor4312 connor4312 added the bug Issue identified by VS Code Team member as probable bug label Jun 3, 2020
@connor4312 connor4312 added this to the May 2020 milestone Jun 3, 2020
connor4312 added a commit to microsoft/vscode-js-debug that referenced this issue Jun 3, 2020
@Tyriar
Copy link
Member

Tyriar commented Jun 5, 2020

This looks wrong now:

image

Shouldn't pwa-node get the same treatment as node-terminal?

@connor4312
Copy link
Member

The difficulty with doing that is that would require the user to have either js-debug or the built-in node extension installed, as registering a debug adapter descriptor factory for a debug type twice will fail. Requiring that would make opting-in harder / remove the possibility of having the usePreview flag.

@Tyriar
Copy link
Member

Tyriar commented Jun 5, 2020

@connor4312 would these eventually change to just node?

@connor4312
Copy link
Member

Yea

@Tyriar Tyriar added the verified Verification succeeded label Jun 5, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jul 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants