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

Look in parent directories for nuget.config files #7342

Conversation

jmarolf
Copy link
Contributor

@jmarolf jmarolf commented May 17, 2023

technically nuget.config files don't need to be next to project or solution files. Nuget traverses the parent directories as well. This change allows us to look for the nuget.config files in the same way the nuget client does

@github-actions github-actions bot added the L: dotnet:nuget NuGet packages via nuget or dotnet label May 17, 2023
@jmarolf jmarolf force-pushed the dev/jmarolf/bugfix/look-for-nugetconfig-in-parent-folder branch 4 times, most recently from 89032ec to 6bc9c17 Compare May 18, 2023 15:50
@jmarolf jmarolf marked this pull request as ready for review May 18, 2023 15:55
@jmarolf jmarolf requested a review from a team as a code owner May 18, 2023 15:55
@jurre
Copy link
Member

jurre commented Jun 5, 2023

Thanks! The code change looks good to me and the new functionality makes sense. We'll need to hold off with merging this for a bit as we're prioritizing some stability work internally. I'll update here once we're ready to accept this.

@jurre jurre force-pushed the dev/jmarolf/bugfix/look-for-nugetconfig-in-parent-folder branch from 6bc9c17 to 6597d50 Compare June 20, 2023 07:59
@jurre jurre merged commit fcfbe0b into dependabot:main Jun 20, 2023
83 checks passed
brettfo pushed a commit to brettfo/dependabot-core that referenced this pull request Oct 11, 2023
Public pull request is here: dependabot#7342

Related work items: #2064024
brettfo pushed a commit to brettfo/dependabot-core that referenced this pull request Oct 11, 2023
…ok-for-nugetconfig-in-parent-folder

Look in parent directories for nuget.config files
brettfo pushed a commit to brettfo/dependabot-core that referenced this pull request Oct 11, 2023
Public pull request is here: dependabot#7342

Related work items: #2064024
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
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants