DependabotSemver Service Test and Badge Failing #7267
Labels
keep-service-tests-green
Related to fixing failing tests of the services
needs-discussion
A consensus is needed to move forward
service-badge
Accepted and actionable changes, features, and bugs
🕚 When did the problem start?
~5 days ago, November 8th
📷 Live badge
https://img.shields.io/dependabot/semver/bundler/puma
🔧 Is the live badge working?
🔗 CircleCI link
https://app.circleci.com/pipelines/github/badges/daily-tests/1321/workflows/0b53d87a-2480-4b3e-bb3a-22f88e3d9938/jobs/2223
🐞 Stack trace
💡 Possible solution
It seems that the API contract has changed on us and now requires two additional parameters. Obviously our existing requests don't send those new parameters so we're getting 400 responses with complaints about bad requests.
The API now wants both a
previous-version
andnew-version
to be provided in the query params. Part of me wonders if this amounts to an incredibly similar, but technically new data point since it's now between explicit pairs (and I don't know if the prior/original percentage reflected an aggregate for the package, ranges, etc.)https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=react-select&package-manager=npm_and_yarn&previous-version=4.3.1&new-version=5.0.0
I think we can try to follow up with GitHub to see if the old API exists in any shape or form, but if not, then I'd lean towards deprecating the current badge. We could create a "new" badge that has new required route parameters to account for the previous and new versions, though I'm not sure how much utility that will provide for our users given what seems to be an exact version pinning
The text was updated successfully, but these errors were encountered: