Repository navigation
Fix #14576: Add updatePackageLastAccessTime to valid config keys list - #6809
Conversation
Co-authored-by: nkolev92 <[email protected]>
|
@copilot write-up a pull request description following the https://github.com/NuGet/NuGet.Client/blob/dev/.github/PULL_REQUEST_TEMPLATE.md template and paste it as a comment.. |
Here's the PR description following the template: BugFixes: NuGet/Home#14576 DescriptionThe Changes:
This minimal fix enables users to successfully run: dotnet nuget config set updatePackageLastAccessTime truePR Checklist
|
|
For some reason the auto rotation did nothing, so I just assign an extra person. |
Bug
Fixes: NuGet/Home#14576
Description
The
updatePackageLastAccessTimeconfiguration setting was added in PR #4222, but it was not included in the list of valid config keys returned byConfigurationConstants.GetConfigKeys(). This causeddotnet nuget config set updatePackageLastAccessTime trueto fail with the error:'updatePackageLastAccessTime' is not a valid config key in config section.Changes:
UpdatePackageLastAccessTimeto the list returned byConfigurationConstants.GetConfigKeys()inConfigurationContants.cs[InlineData("updatePackageLastAccessTime", "true")]toXPlatConfigTests.ConfigSetCommand_WithConfigFileArg_AddsSettingto verify the config key can be set successfullyThis minimal fix enables users to successfully run:
PR Checklist