Skip to content

docs: recommend pinning SDK version when using NuGet package lock files#54846

Draft
adegeo with Copilot wants to merge 2 commits into
mainfrom
copilot/update-package-lock-docs
Draft

docs: recommend pinning SDK version when using NuGet package lock files#54846
adegeo with Copilot wants to merge 2 commits into
mainfrom
copilot/update-package-lock-docs

Conversation

Copilot AI commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

The package lock files docs didn't mention that SDK version pinning is also required for truly reproducible restores—updating the SDK can change the dependency graph and silently invalidate the lock file.

Changes

  • docs/core/install/upgrade.md — Package lock files section
    • Adds an [!IMPORTANT] callout recommending a global.json with rollForward: disable alongside lock file usage
    • Includes a minimal JSON example for the recommended config
    • Links back to the existing "Control SDK version with global.json" section on the same page and to the upstream tracking issues (dotnet/aspnetcore#65061, dotnet/sdk#48795)

The global-json.md rollForward table already has a reverse footnote pointing at this section; these two pages now cross-reference each other symmetrically.

Recommended config when using lock files:

{
  "sdk": {
    "version": "9.0.100",
    "rollForward": "disable"
  }
}

Copilot AI review requested due to automatic review settings July 21, 2026 16:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot AI changed the title [WIP] Update package lock documentation to recommend pinning SDK version docs: recommend pinning SDK version when using NuGet package lock files Jul 21, 2026
Copilot AI requested a review from adegeo July 21, 2026 16:34
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.

Package lock docs should mention pinning the SDK version as a recommendation

3 participants