Skip to content

Commit 05d72c0

Browse files
committed
Referencing parameters using nameof to aid in discovery and potential renames in the future
1 parent 8acca4b commit 05d72c0

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/Commands/Lists/SetList.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -273,12 +273,12 @@ protected override void ExecuteCmdlet()
273273
{
274274
if (!ParameterSpecified(nameof(MajorVersions)) || !ParameterSpecified(nameof(ExpireVersionsAfterDays)))
275275
{
276-
throw new PSArgumentException("You must specify a value for ExpireVersionsAfterDays and MajorVersions");
276+
throw new PSArgumentException($"You must specify a value for {nameof(ExpireVersionsAfterDays)} and {nameof(MajorVersions)}", nameof(ExpireVersionsAfterDays));
277277
}
278278

279279
if (!ParameterSpecified(nameof(MinorVersions)) && list.EnableMinorVersions)
280280
{
281-
throw new PSArgumentException("You must specify a value for MinorVersions if it is enabled.");
281+
throw new PSArgumentException($"You must specify a value for {nameof(MinorVersions)} if it is enabled.", nameof(MinorVersions));
282282
}
283283

284284
if (ExpireVersionsAfterDays == 0)
@@ -291,7 +291,7 @@ protected override void ExecuteCmdlet()
291291
}
292292
else
293293
{
294-
throw new PSArgumentException("You must specify ExpireVersionsAfterDays to be 0 for NoExpiration or greater equal 30 for ExpireAfter.");
294+
throw new PSArgumentException($"You must specify {nameof(ExpireVersionsAfterDays)} to be 0 for NoExpiration or greater equal 30 for ExpireAfter.", nameof(ExpireVersionsAfterDays));
295295
}
296296
}
297297

@@ -302,7 +302,7 @@ protected override void ExecuteCmdlet()
302302
{
303303
if (list.VersionPolicies.DefaultTrimMode == VersionPolicyTrimMode.AutoExpiration)
304304
{
305-
throw new PSArgumentException("The parameter ExpireVersionsAfterDays can't be set when AutoExpiration is enabled");
305+
throw new PSArgumentException($"The parameter {nameof(ExpireVersionsAfterDays)} can't be set when AutoExpiration is enabled");
306306
}
307307

308308
list.VersionPolicies.DefaultExpireAfterDays = (int)ExpireVersionsAfterDays;
@@ -313,7 +313,7 @@ protected override void ExecuteCmdlet()
313313
{
314314
if (list.VersionPolicies.DefaultTrimMode == VersionPolicyTrimMode.AutoExpiration)
315315
{
316-
throw new PSArgumentException("The parameter MajorVersions can't be set when AutoExpiration is enabled");
316+
throw new PSArgumentException($"The parameter {nameof(MajorVersions)} can't be set when AutoExpiration is enabled", nameof(MajorVersions));
317317
}
318318

319319
list.MajorVersionLimit = (int)MajorVersions;
@@ -324,7 +324,7 @@ protected override void ExecuteCmdlet()
324324
{
325325
if (list.VersionPolicies.DefaultTrimMode == VersionPolicyTrimMode.AutoExpiration)
326326
{
327-
throw new PSArgumentException("The parameter MinorVersions can't be set when AutoExpiration is enabled");
327+
throw new PSArgumentException($"The parameter {nameof(MinorVersions)} can't be set when AutoExpiration is enabled", nameof(MinorVersions));
328328
}
329329

330330
list.MajorWithMinorVersionsLimit = (int)MinorVersions;

0 commit comments

Comments
 (0)