Skip to content

Fix pipeline: pin exact SDK version and add PR trigger#123

Merged
WilliamsJason merged 1 commit into
mainfrom
users/jaswill/fix-pipeline-sdk-pinning
May 14, 2026
Merged

Fix pipeline: pin exact SDK version and add PR trigger#123
WilliamsJason merged 1 commit into
mainfrom
users/jaswill/fix-pipeline-sdk-pinning

Conversation

@WilliamsJason
Copy link
Copy Markdown
Collaborator

@WilliamsJason WilliamsJason commented May 14, 2026

Problem

The CI build has been failing because global.json used rollForward: latestPatch, allowing the CI agent to install a newer SDK patch than the one used to generate lock files. The newer SDK ships different implicit package versions, breaking --locked-mode NuGet restore.

Changes

  • global.json: Changed rollForward from latestPatch to disable to ensure CI uses the exact SDK version the lock files were generated with.
  • build-pipeline.yml: Added pr trigger for main branch.

@WilliamsJason WilliamsJason force-pushed the users/jaswill/fix-pipeline-sdk-pinning branch from dffd232 to f7f3c05 Compare May 14, 2026 20:45
- Change global.json rollForward from 'latestPatch' to 'disable' to
  prevent implicit package version drift that breaks locked-mode restore.
- Add pr trigger for main branch to build-pipeline.yml.
- Update dotnet.yml and codeql.yml GitHub Actions to use global.json
  instead of floating 10.0.x, and add --locked-mode to restore.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@WilliamsJason WilliamsJason force-pushed the users/jaswill/fix-pipeline-sdk-pinning branch from f7f3c05 to aaf44aa Compare May 14, 2026 20:47
@RafaelHinojosa RafaelHinojosa self-requested a review May 14, 2026 20:58
@WilliamsJason WilliamsJason merged commit e95e6f3 into main May 14, 2026
7 checks passed
@WilliamsJason WilliamsJason deleted the users/jaswill/fix-pipeline-sdk-pinning branch May 14, 2026 21:22
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.

2 participants