Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ensure proper casing for NuGet.Config prior to any operations being performed #11659

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

brettfo
Copy link
Contributor

@brettfo brettfo commented Feb 21, 2025

NuGet only allows three options for the config file naming:

  • nuget.config
  • NuGet.config
  • NuGet.Config

If a code base is primarily developed on a case-insensitive file system like Windows, a config file named Nuget.config is fine, but dependabot runs in a Linux container with a case-sensitive file system and in that instance the config file won't be picked up by the NuGet tooling resulting in the default package feed of api.nuget.org.

This PR addresses that by running a script to rename the files prior to any operation being performed.

Fixes #11525.

@github-actions github-actions bot added the L: dotnet:nuget NuGet packages via nuget or dotnet label Feb 21, 2025
@brettfo brettfo force-pushed the dev/brettfo/nuget-config-not-honored branch 4 times, most recently from c0ccf85 to a5a3715 Compare February 21, 2025 22:58
@brettfo brettfo force-pushed the dev/brettfo/nuget-config-not-honored branch from a5a3715 to a686984 Compare February 21, 2025 23:04
@brettfo brettfo marked this pull request as ready for review February 21, 2025 23:39
@brettfo brettfo requested a review from a team as a code owner February 21, 2025 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: dotnet:nuget NuGet packages via nuget or dotnet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Custom nuget-feed no longer used, breaking package updates
3 participants