Skip to content

fix(ci): route NuGet version check through CFS - #3162

Merged
Gavin Barron (gavinbarron) merged 3 commits into
mainfrom
gavinbarron-fix-cfsclean-failures
Aug 5, 2026
Merged

fix(ci): route NuGet version check through CFS#3162
Gavin Barron (gavinbarron) merged 3 commits into
mainfrom
gavinbarron-fix-cfsclean-failures

Conversation

@gavinbarron

@gavinbarron Gavin Barron (gavinbarron) commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary

  • route the published-package version check through the authenticated Central Feed Service
  • remove the direct public NuGet API request that triggers CFSClean
  • pass the generated nuget.config into the validation script

Validation

  • exercised dotnet package search --configfile against Microsoft.Graph
  • confirmed the version comparison still succeeds
Microsoft Reviewers: Open in CodeFlow

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 03d194c7-5fba-4802-a182-3b3ef5c5c0ab
Replace the direct public NuGet API request with dotnet package search using the authenticated Central Feed Service configuration.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 03d194c7-5fba-4802-a182-3b3ef5c5c0ab
Copilot AI lite review requested due to automatic review settings August 5, 2026 01:05
@gavinbarron
Gavin Barron (gavinbarron) requested a review from a team as a code owner August 5, 2026 01:05

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.

Pull request overview

This PR updates the CI “published-package version” validation to query package versions via the authenticated Central Feed Service (CFS) using dotnet package search and a generated nuget.config, eliminating the direct unauthenticated call to the public NuGet registration API.

Changes:

  • Add a mandatory -nugetConfigPath parameter to the validation script and use it for package search.
  • Replace Invoke-RestMethod (NuGet registration API) with dotnet package search --format json.
  • Wire the generated nuget.config path into the Azure Pipelines validation step.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
scripts/ValidateUpdatedNugetVersion.ps1 Switches version lookup from NuGet registration API to dotnet package search using an authenticated nuget.config.
.azure-pipelines/ci-build.yml Passes the generated nuget.config path into the validation script invocation.

Comment thread scripts/ValidateUpdatedNugetVersion.ps1 Outdated
@gavinbarron
Gavin Barron (gavinbarron) enabled auto-merge (squash) August 5, 2026 01:12
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 03d194c7-5fba-4802-a182-3b3ef5c5c0ab
@gavinbarron
Gavin Barron (gavinbarron) merged commit 8c6b248 into main Aug 5, 2026
6 checks passed
@gavinbarron
Gavin Barron (gavinbarron) deleted the gavinbarron-fix-cfsclean-failures branch August 5, 2026 02:09
This was referenced Aug 6, 2026
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.

3 participants