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

Update code.cmd #209315

Closed
wants to merge 31 commits into from
Closed

Update code.cmd #209315

wants to merge 31 commits into from

Conversation

longtsing
Copy link

@longtsing longtsing commented Apr 2, 2024

Fixed the issue where the cmd window remains when executing the code command under cmd to open vscode.

The issue :
#210876
#70139

The reason for the bug is that when cmd opens vscode in the foreground, it will get stuck in the event loop of electron.
So the cmd window can only exit automatically after vscode exits.
The solution is to open a new process and open vsocode in the background.

Fixed the issue where the cmd window remains when executing the code command under cmd to open vscode.

The issue :
microsoft#70139
Copy link
Author

@longtsing longtsing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add "START /B" to start a new process in the background to start vscode. The empty string is to fix the problem of the pop-up window prompting to select the js interpreter in the windows system.

Copy link
Author

@longtsing longtsing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It work in windows10

@longtsing
Copy link
Author

@DonJayamanne May I have the Honor to contribute for the vscode?Please.

@DonJayamanne
Copy link
Contributor

@longtsing The issue has been closed, please create a new issue so we know what you are trying to address.

@longtsing
Copy link
Author

@longtsing The issue has been closed, please create a new issue so we know what you are trying to address.
@DonJayamanne

the new issue is these:
#210876

@longtsing
Copy link
Author

@DonJayamanne hello?

@DonJayamanne DonJayamanne removed their assignment Apr 28, 2024
@DonJayamanne
Copy link
Contributor

@longtsing I would suggest you first wait for the issue to get triaged by @roblourens
Once the issue has been triaged, then the PR can be accepted.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

我试了一下,windows10上确定能用,不过 /b 参数加不加都一样

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

是的,不过加 /B 参数是为了保持兼容性,windows 10下可能加不加都一样,但是其他的版本的 windows 加个 /b 应该好一些。

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

就等给 issue 分类,然后合并提交了。

@longtsing
Copy link
Author

@roblourens

@roblourens
Copy link
Member

See #66750

@roblourens roblourens closed this May 13, 2024
@microsoft microsoft locked and limited conversation to collaborators Jun 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants