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

[NuGet] Add Analyze command to NuGetUpdater #9493

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

JoeRobich
Copy link
Collaborator

@JoeRobich JoeRobich commented Apr 12, 2024

This PR will add an analyze command to the NuGetUpdater. The ruby update_checker will switch to using the results of this command to answer questions about updating a particular dependency.

TODO:

  • Get smoke tests passing or update them if necessary

@github-actions github-actions bot added the L: dotnet:nuget NuGet packages via nuget or dotnet label Apr 12, 2024
@james-asebp
Copy link

james-asebp commented Apr 15, 2024

Is this change likely to help resolve (or get closer to resolving) #8578 (comment)?

@JoeRobich
Copy link
Collaborator Author

Is this change likely to help resolve (or get closer to resolving) #8578 (comment)?

This will be the first step towards addressing that issue. We will have additional information which will be used when performing the actual updates. There will be follow up changes where we calculate update plans for each project that will be more precise than the current updates.

@JoeRobich JoeRobich force-pushed the jorobich/NativeUpdateChecker branch from b91ce6e to dd839ba Compare April 18, 2024 21:56
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.

None yet

2 participants