Skip to content

Add HelpName for --project and --launch-profile arguments of RunCommand #49418

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

Merged
merged 1 commit into from
Jun 16, 2025

Conversation

akoeplinger
Copy link
Member

Contributes to #48817

/cc @baronfel

@Copilot Copilot AI review requested due to automatic review settings June 16, 2025 17:18
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds help name properties for the --project and --launch-profile options of RunCommand and updates related snapshots and localization resource files accordingly.

  • Updated test snapshot to reflect new identifiers (PROJECT_PATH and LAUNCH_PROFILE).
  • Modified RunCommandParser.cs to assign HelpName values from CliCommandStrings.
  • Updated multiple .xlf and .resx files with new trans-unit entries for the help names.

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated no comments.

File Description
test/dotnet.Tests/CompletionTests/snapshots/zsh/DotnetCliSnapshotTests.VerifyCompletions.verified.zsh Updated snapshot entries to reflect help name changes
src/Cli/dotnet/Commands/xlf/*.xlf Added new trans-units defining LAUNCH_PROFILE and PROJECT_PATH for help names
src/Cli/dotnet/Commands/Run/RunCommandParser.cs Updated Option definitions to include HelpName properties
src/Cli/dotnet/Commands/CliCommandStrings.resx Added help name data entries for launch profile and project options

Copy link
Member

@baronfel baronfel left a comment

Choose a reason for hiding this comment

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

THANK YOU SO MUCH

@akoeplinger
Copy link
Member Author

Failure is an unrelated issue related to Docker Hub request limit.

@akoeplinger akoeplinger merged commit 8ad8818 into dotnet:main Jun 16, 2025
28 of 31 checks passed
@akoeplinger akoeplinger deleted the helpname branch June 16, 2025 18:41
akoeplinger added a commit to akoeplinger/sdk that referenced this pull request Jun 16, 2025
Follow-up to dotnet#48817 and dotnet#49418.

There were only a couple commands where the option HelpName was overridden, make those explicit instead and add a test that verifies all options comply.
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.

3 participants