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

regression test failure with 4.9 release #729

Closed
chenrui333 opened this issue Apr 7, 2024 · 2 comments · Fixed by #730
Closed

regression test failure with 4.9 release #729

chenrui333 opened this issue Apr 7, 2024 · 2 comments · Fixed by #730
Labels

Comments

@chenrui333
Copy link
Contributor

👋 looks like something got changed that buku --nostdin --nc --update no longer update index. This was returned before

Title: [GitHub - Homebrew/brew: 🍺 The missing package manager for macOS (or Linux)]
Index 1: updated

This caused some regression test failure, see in https://github.com/Homebrew/homebrew-core/actions/runs/8586780682/job/23529673964

relates to Homebrew/homebrew-core#168219

@LeXofLeviafan
Copy link
Collaborator

Well that's an unpleasant one… It's a bug in the data passed by CLI; I don't use full DB mass fetches (which sounds like a good way to get network-banned by IP as a bot by some servers 😅) and our tests don't cover CLI itself so I didn't catch it before the release.

…A bug like this sounds like a good reason for a hotfix release 😅

jarun added a commit that referenced this issue Apr 8, 2024
[#729] fixed url value passed to update_rec() from CLI
@jarun jarun closed this as completed in #730 Apr 8, 2024
@jarun
Copy link
Owner

jarun commented Apr 8, 2024

A bug like this sounds like a good reason for a hotfix release 😅

We'll wait for a while on this. Since this release is made after more than a year, there might be other regressions. Let's get some feedback.

@github-actions github-actions bot locked and limited conversation to collaborators May 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants