Skip to content

Commit 3b917c6

Browse files
Copilotbaronfel
authored andcommitted
Use single format string with placeholders instead of separate resource strings
Co-authored-by: baronfel <[email protected]>
1 parent fea93b8 commit 3b917c6

17 files changed

+35
-103
lines changed

src/Cli/dotnet/CliStrings.resx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -817,10 +817,7 @@ The default is 'false.' However, when targeting .NET 7 or lower, the default is
817817
<comment>{Locked="dotnet workload update"}</comment>
818818
</data>
819819
<data name="PackageIdentityArgumentDescription" xml:space="preserve">
820-
<value>Package reference in the form of a package identifier like 'Newtonsoft.Json' or package identifier and version separated by '@' like '[email protected]'.</value>
821-
</data>
822-
<data name="ToolPackageIdentityArgumentDescription" xml:space="preserve">
823-
<value>Package reference in the form of a package identifier like 'dotnetsay' or package identifier and version separated by '@' like '[email protected]'.</value>
820+
<value>Package reference in the form of a package identifier like '{0}' or package identifier and version separated by '@' like '{0}@{1}'.</value>
824821
</data>
825822
<data name="PackageIdentityArgumentIdOrVersionIsNull" xml:space="preserve">
826823
<value>Package reference id and version must not be null.</value>

src/Cli/dotnet/Commands/Tool/Install/ToolInstallCommandParser.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ namespace Microsoft.DotNet.Cli.Commands.Tool.Install;
1010

1111
internal static class ToolInstallCommandParser
1212
{
13-
public static readonly Argument<PackageIdentityWithRange> PackageIdentityArgument = CommonArguments.RequiredPackageIdentityArgument(CliStrings.ToolPackageIdentityArgumentDescription);
13+
public static readonly Argument<PackageIdentityWithRange> PackageIdentityArgument = CommonArguments.RequiredPackageIdentityArgument("dotnetsay", "2.1.7");
1414

1515
public static readonly Option<string> VersionOption = new("--version")
1616
{

src/Cli/dotnet/Commands/Tool/Update/ToolUpdateCommandParser.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ namespace Microsoft.DotNet.Cli.Commands.Tool.Update;
99

1010
internal static class ToolUpdateCommandParser
1111
{
12-
public static readonly Argument<PackageIdentityWithRange?> PackageIdentityArgument = CommonArguments.OptionalPackageIdentityArgument(CliStrings.ToolPackageIdentityArgumentDescription);
12+
public static readonly Argument<PackageIdentityWithRange?> PackageIdentityArgument = CommonArguments.OptionalPackageIdentityArgument("dotnetsay", "2.1.7");
1313

1414
public static readonly Option<bool> UpdateAllOption = ToolAppliedOption.UpdateAllOption;
1515

src/Cli/dotnet/CommonArguments.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -12,23 +12,23 @@ namespace Microsoft.DotNet.Cli
1212
internal class CommonArguments
1313
{
1414
public static DynamicArgument<PackageIdentityWithRange?> OptionalPackageIdentityArgument() =>
15-
OptionalPackageIdentityArgument(CliStrings.PackageIdentityArgumentDescription);
15+
OptionalPackageIdentityArgument("Newtonsoft.Json", "13.0.3");
1616

17-
public static DynamicArgument<PackageIdentityWithRange?> OptionalPackageIdentityArgument(string description) =>
17+
public static DynamicArgument<PackageIdentityWithRange?> OptionalPackageIdentityArgument(string examplePackage, string exampleVersion) =>
1818
new("packageId")
1919
{
20-
Description = description,
20+
Description = string.Format(CliStrings.PackageIdentityArgumentDescription, examplePackage, exampleVersion),
2121
CustomParser = (ArgumentResult argumentResult) => ParsePackageIdentityWithVersionSeparator(argumentResult.Tokens[0]?.Value),
2222
Arity = ArgumentArity.ZeroOrOne,
2323
};
2424

2525
public static DynamicArgument<PackageIdentityWithRange> RequiredPackageIdentityArgument() =>
26-
RequiredPackageIdentityArgument(CliStrings.PackageIdentityArgumentDescription);
26+
RequiredPackageIdentityArgument("Newtonsoft.Json", "13.0.3");
2727

28-
public static DynamicArgument<PackageIdentityWithRange> RequiredPackageIdentityArgument(string description) =>
28+
public static DynamicArgument<PackageIdentityWithRange> RequiredPackageIdentityArgument(string examplePackage, string exampleVersion) =>
2929
new("packageId")
3030
{
31-
Description = description,
31+
Description = string.Format(CliStrings.PackageIdentityArgumentDescription, examplePackage, exampleVersion),
3232
CustomParser = (ArgumentResult argumentResult) => ParsePackageIdentityWithVersionSeparator(argumentResult.Tokens[0]?.Value)!.Value,
3333
Arity = ArgumentArity.ExactlyOne,
3434
};

src/Cli/dotnet/xlf/CliStrings.cs.xlf

Lines changed: 2 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/Cli/dotnet/xlf/CliStrings.de.xlf

Lines changed: 2 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/Cli/dotnet/xlf/CliStrings.es.xlf

Lines changed: 2 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/Cli/dotnet/xlf/CliStrings.fr.xlf

Lines changed: 2 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/Cli/dotnet/xlf/CliStrings.it.xlf

Lines changed: 2 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/Cli/dotnet/xlf/CliStrings.ja.xlf

Lines changed: 2 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)