Skip to content

Add authenication and nuget feed cofig into yml for fetch and restore… - #1100

Open
v-ochoudhary wants to merge 1 commit into
mainfrom
users/v-ochoudhary/fix-pr-build-checks
Open

v-ochoudhary wants to merge 1 commit into
mainfrom
users/v-ochoudhary/fix-pr-build-checks

Conversation

@v-ochoudhary

Copy link
Copy Markdown

Details

This pull request updates the build pipeline configuration to improve NuGet authentication and package restore reliability. The changes ensure all build jobs use a consistent nuget.config file and explicitly authenticate to the NuGet feeds before restoring packages.

Key changes include:

NuGet Authentication and Restore Improvements:

  • Added the NuGetAuthenticate@1 task before all NuGet restore steps across prbuild.yml, signedbuild.yml, and check-dependencies.yml to ensure authenticated access to private feeds. [1] [2] [3] [4] [5] [6]
  • Updated all NuGetCommand@2 and DotNetCoreCLI@2 restore tasks to use a centralized NuGet configuration file (src/nuget.config) by setting the feedsToUse: config and nugetConfigPath inputs. This ensures consistent and secure package source usage. [1] [2] [3] [4] [5] [6] [7]

Configuration and Policy Updates:

  • Added a new src/nuget.config file specifying the a11y-insights-public package source for all restores.
  • Set the networkIsolationPolicy to Permissive,CFSClean in build pipeline templates to align with updated security/networking requirements. [1] [2]

These changes help standardize package management and improve build reliability across all pipelines.…

Motivation

Fix PR checks failed due to nuget package is not restore.

Pull request checklist

  • Addresses an existing issue: #0000

@v-ochoudhary
v-ochoudhary requested a review from a team as a code owner September 16, 2026 10:31
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@v-ochoudhary
v-ochoudhary requested a lite review from Copilot September 16, 2026 10:32
@v-ochoudhary

Copy link
Copy Markdown
Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Clearing existing sources may prevent required packages from restoring unless the configured feed provides the necessary upstreams.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

This PR standardizes authenticated NuGet restores across Azure Pipelines.

Changes:

  • Adds centralized src/nuget.config.
  • Authenticates NuGet feeds before restores.
  • Configures restore tasks to use the shared configuration.
  • Updates network isolation settings.
File summaries
File Description
src/nuget.config Defines the shared package source.
build/signedbuild.yml Updates authenticated signed-build restores.
build/prbuild.yml Updates authenticated PR-build restores.
build/check-dependencies.yml Updates authenticated dependency checks.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/nuget.config
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