-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Support NuGet lockfiles (Updated) #9678
Conversation
The linting failure for fetch_files is because your changes pushed the complexity over the limit. fetched_files = []
fetched_files += project_files
fetched_files += directory_build_files
fetched_files += imported_property_files
fetched_files += packages_config_files
fetched_files += nuget_config_files
fetched_files += packages_lock_files
fetched_files << global_json if global_json
fetched_files << dotnet_tools_json if dotnet_tools_json
fetched_files << packages_props if packages_props with this: fetched_files = [
*project_files,
*directory_build_files,
*imported_property_files,
*packages_config_files,
*nuget_config_files,
*packages_lock_files,
global_json,
dotnet_tools_json,
packages_props
].compact and that should reduce the complexity. |
I don't know why the test failed. It was successful when I tested it locally. |
nuget/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Updater/UpdaterWorker.cs
Outdated
Show resolved
Hide resolved
nuget/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Updater/UpdaterWorker.cs
Outdated
Show resolved
Hide resolved
The |
@na1307 - do you need help with this? I am very keen to get this merged. |
Hey @jonjanego, would you be able to have someone from the team take a look at this to evaluate next steps? |
nuget/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Updater/LockFileUpdater.cs
Outdated
Show resolved
Hide resolved
nuget/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Updater/LockFileUpdater.cs
Outdated
Show resolved
Hide resolved
thanks @na1307 for the contribution! |
Actually huge that this work is now completed |
This seems to fail if the
The project path should be quoted in the execution to make sure it's passed completely. |
dependabot-core/nuget/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Updater/LockFileUpdater.cs Line 21 in ddb9722
Oh, I think I forgot the quotation marks. |
@na1307 I've forked the project and found more issues where paths are not quoted. I'm working on a fix. |
PR here: #10676 |
Looks like (from the logs) that Dependabot now generates lock file updates - great! But sadly at the same time it started failing to create the PR for us, GitHub responds with 400 log file excerpt
|
Fix #1303
Since #6031 was last updated, Dependabot NuGet logic has been rewritten as C#, so I've put this PR up.
I'm familiar with C#, but not with Ruby. I don't know how to add a test for this. Any advice is welcome.