Skip to content

Add nuget.config with nuget.org package source#101

Merged
mattwar merged 1 commit into
microsoft:mainfrom
tomer-w:add-nuget-config
May 5, 2026
Merged

Add nuget.config with nuget.org package source#101
mattwar merged 1 commit into
microsoft:mainfrom
tomer-w:add-nuget-config

Conversation

@tomer-w

@tomer-w tomer-w commented May 5, 2026

Copy link
Copy Markdown
Contributor

Problem

Building the server project (\src/Server) fails on machines where nuget.org is not configured as a machine-level NuGet package source, since packages like \Microsoft.Azure.Kusto.Data\ and \StreamJsonRpc\ cannot be found.

Solution

Add a repo-level
uget.config\ that explicitly configures nuget.org as the package source. Uses <clear />\ to ensure a clean, predictable set of sources regardless of machine configuration.

Validation

  • Reverted machine-level nuget.org source to confirm the repo-level config is sufficient
  • Verified \dotnet restore\ and \dotnet build\ succeed using only the repo-level config

Ensures all contributors have the correct NuGet package source
configured without relying on machine-level settings.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 5, 2026 09:57

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

Adds a repository-level nuget.config to make NuGet package source configuration deterministic across machines, preventing restore/build failures for the server project when nuget.org isn’t configured at the machine level.

Changes:

  • Introduces a root nuget.config that clears existing sources and explicitly adds nuget.org.
  • Ensures dotnet restore / dotnet build can succeed without relying on developer machine NuGet source setup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mattwar mattwar merged commit b77c54f into microsoft:main May 5, 2026
8 of 9 checks passed
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