-
Notifications
You must be signed in to change notification settings - Fork 136
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
Introduction of unwanted changes to launch.json #537
Comments
@bersbersbers to help us investigate, would you please provide more info on the actual contents that are added to the |
Today, I was able to repro this by selecting and choosing Java - hope this helps: {
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [
{
"type": "java",
"name": "Launch Current File",
"request": "launch",
"mainClass": "${file}"
}
]
} |
Created an issue at VS Code to track it since our debugger extension is reusing VS Code API to read and update launch.json. |
Thanks! I have to admit that when I tested briefly this morning, I only verified that a launch config is added that way - but in fact, the comments from |
microsoft/vscode#110295 has repro steps, also showing why #537 (comment) does not repro the issue. Follow microsoft/vscode#29453 for progress, Thanks! |
Installation/use of https://marketplace.visualstudio.com/items?itemName=vscjava.vscode-java-pack reformats the
launch.json
file when adding its launch configs, thereby removing trailing commas (no big deal) and comments (BIG DEAL). I cannot say for sure when it happened, but it was around installation and short running/debugging a hello-world-example in Java. If people don't have their latest changes to comments inlaunch.json
committed to version control, they are lost.Environment:
Step to Reproduce:
launch.json
having commentsExpected Behavior:
launch.json
Actual Behavior:
launch.json
The text was updated successfully, but these errors were encountered: