Skip to content

Conversation

@claudiaregio
Copy link
Contributor

@claudiaregio claudiaregio commented Jan 14, 2026

This PR should update command prefixes from "CSharp" to "C#" and have one command named "CSharp: Commands now start with C#" and when clicked it should open a link in the browser (aka.ms/new-csharp-commands)

DRAFT as this should be merged into Feb Pre-release 1

@claudiaregio claudiaregio requested a review from a team as a code owner January 14, 2026 20:12
@claudiaregio claudiaregio marked this pull request as draft January 14, 2026 20:14
Comment on lines +1805 to +1808
"command": "csharp.openUrlInBrowser",
"title": "%commands.openUrlInBrowser%",
"category": "CSharp"
},
Copy link
Member

Choose a reason for hiding this comment

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

indentation seems wrong here (as compared to the other command definitions)

],
"commands": [
{
"command": "csharp.openUrlInBrowser",
Copy link
Member

Choose a reason for hiding this comment

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

Possibly make the commandId here more specific? e.g. csharp.showNewCommandDocs (or similar), since it is opening a specific page and not any URL.

"commands": [
{
"command": "csharp.openUrlInBrowser",
"title": "%commands.openUrlInBrowser%",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"title": "%commands.openUrlInBrowser%",
"title": "%command.openUrlInBrowser%",

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.

4 participants