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
fix(nuget): Sort api response before picking projectUrl #23090
fix(nuget): Sort api response before picking projectUrl #23090
Conversation
…epage Use the existing NuGet versioning module to sort versions Add test including anonymised actual response from Azure DevOps package feed for nunit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reduce test fixtures to a minimum
I think this is about as terse as I can get them without breaking anything. edit: actually I could also remove some intermediate versions of nunit from the registration API response. |
lib/modules/datasource/nuget/__fixtures__/azure_devops/nunit/nuspec.xml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise LGTM
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
🎉 This PR is included in version 37.20.2 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exelente
…23090) Co-authored-by: Rhys Arkins <rhys@arkins.net> Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
Changes
Sort NuGet API responses using the versioning module before finding the latest projectUrl
Add test including response from Azure DevOps package feed for nunit
Context
Closes #22887
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: