Convert launch.json properties to camelCase
#21
Conversation
README.md
Outdated
|
|
||
| ### Updating the GitHub release page | ||
| After any PR has been merged, you need to consider if it is necessary to update the `.vsix` in the repository's [release page](https://github.com/probe-rs/vscode/releases). | ||
| * For minor fixes and changes, just update the asset for the latest release. |
There was a problem hiding this comment.
I think we should always create a new release, updating an existing release can be very confusing.
There was a problem hiding this comment.
I can align with that. Ideally I'd like bors to compile and create the release, but I don't know enough about bors to automate that.
README.md
Outdated
| ```bash | ||
| code --install-extension amodio.tsl-problem-matcher | ||
| ``` | ||
| - Install the extensions VS Code recommends |
There was a problem hiding this comment.
I like doing this on the command line, since it is also used in the previous steps. Maybe we can just say something like "Install the recommended extensions in VS Code, or run the following command line: "?
There was a problem hiding this comment.
That would work, but then we'd have to update this README every time we have some new extension required (like the discussions about enforcing code formatting. I have a suggestion that might help both of us. Let me push into this PR and you can see if you are OK with it.
885: Convert `launch.json` properties to camelCase r=Yatekii a=noppej *Note:* The changes in this PR is dependent on being synchronized with [probe-rs/vscode PR #21](probe-rs/vscode#21) *Changes* * Convert all existing `launch.json` properties to camelCase * Updates to CHANGELOG Co-authored-by: JackN <[email protected]>
|
Build succeeded: |
Note: The changes in this PR is dependent on being synchronized with probe-rs PR #885
Changes
launch.jsonproperties to camelCase