Skip to content

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

Closed
tomer-w wants to merge 1 commit into
microsoft:mainfrom
tomer-w:add-nuget-config
Closed

Add nuget.config with nuget.org package source#100
tomer-w wants to merge 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:53
@tomer-w tomer-w closed this May 5, 2026

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 aims to make restores/builds deterministic by adding a repository-level nuget.config that defines the NuGet package sources needed to build the server project without relying on machine-level configuration.

Changes:

  • Adds a new repo-level nuget.config.
  • Clears any machine/user-defined package sources and configures a single package source entry.

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

Comment thread nuget.config
<?xml version="1.0" encoding="utf-8"?>
<configuration>
<packageSources>
<clear />
Comment thread nuget.config
<configuration>
<packageSources>
<clear />
<add key="Microsoft Visual Studio Offline Packages" value="C:\Program Files (x86)\Microsoft SDKs\NuGetPackages\" />
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