Skip to content

case-correct web.config and app.config locations when reporting#13851

Merged
brettfo merged 1 commit intomainfrom
dev/brettfo/nuget-app-config-location
Jan 7, 2026
Merged

case-correct web.config and app.config locations when reporting#13851
brettfo merged 1 commit intomainfrom
dev/brettfo/nuget-app-config-location

Conversation

@brettfo
Copy link
Copy Markdown
Contributor

@brettfo brettfo commented Dec 22, 2025

From a manual scan of the telemetry I found that a .csproj might contain <None include="App.config" /> but on disk the file is named app.config (note the differing case.) Since most projects with those files are built on Windows but dependabot runs in a Linux container, we do a case-correction pass on those paths because the intent is obvious.

@github-actions github-actions Bot added the L: dotnet:nuget NuGet packages via nuget or dotnet label Dec 22, 2025
@brettfo brettfo force-pushed the dev/brettfo/nuget-app-config-location branch from 1997351 to 73921c2 Compare December 22, 2025 19:05
@brettfo brettfo marked this pull request as ready for review December 22, 2025 20:08
@brettfo brettfo requested a review from a team as a code owner December 22, 2025 20:08
@brettfo brettfo force-pushed the dev/brettfo/nuget-app-config-location branch from 73921c2 to 183a6be Compare January 7, 2026 18:16
@brettfo brettfo merged commit 7cec680 into main Jan 7, 2026
98 checks passed
@brettfo brettfo deleted the dev/brettfo/nuget-app-config-location branch January 7, 2026 18:37
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.

2 participants