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

fix(datasource/go): support go proxy with abortOnError #29823

Merged
merged 2 commits into from
Jul 15, 2024

Conversation

ahippler
Copy link
Contributor

Changes

Do not fail goproxy version lookup with abortOnError: true.

Context

The goproxy registry api only support listing versions per major version. Renovate tries to fetch the versions for the next major versions to be able to detect major version updated. This is repeated until the major version does not exist and the last requests fails with a 404. With abortOnError enabled the whole lookup throws an error for the last request. The logic to ignore the error does not handle the wrapper error.

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

@ahippler ahippler force-pushed the feature/go-abortOnError branch 2 times, most recently from ce617ad to cfbacf0 Compare June 24, 2024 18:22
@viceice viceice changed the title fix: support go with abortOnError fix(datasource/go): support go proxy with abortOnError Jun 24, 2024
auto-merge was automatically disabled July 15, 2024 08:14

Head branch was pushed to by a user without write access

@ahippler ahippler requested a review from viceice July 15, 2024 08:16
@rarkins rarkins added this pull request to the merge queue Jul 15, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 15, 2024
@viceice viceice added the ci:fulltest Run full test suite on all platforms label Jul 15, 2024
@viceice viceice enabled auto-merge July 15, 2024 17:52
@viceice viceice added this pull request to the merge queue Jul 15, 2024
Merged via the queue into renovatebot:main with commit 38582c1 Jul 15, 2024
51 checks passed
@renovate-release
Copy link
Collaborator

🎉 This issue has been resolved in version 37.431.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ahippler ahippler deleted the feature/go-abortOnError branch July 16, 2024 13:50
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ci:fulltest Run full test suite on all platforms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants