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 install_registry_source_dist_cached on Gentoo #3569

Merged
merged 1 commit into from
May 14, 2024

Conversation

mgorny
Copy link
Contributor

@mgorny mgorny commented May 14, 2024

Summary

Increment the removed file counts in filters
in install_registry_source_dist_cached test, to make it work again on Gentoo. The tested counts were updated
in 9a92a3a, but the filters were not. That said, the respective count increased in Gentoo as well, so adjust both input and output strings. I'm updating Windows as a guesswork, though I suspect that filter may not be necessary anymore, given that CI was passing.

Test Plan

cargo test on Gentoo :-).

Increment the removed file counts in filters
in install_registry_source_dist_cached test, to make it work again
on Gentoo.  The tested counts were updated
in 9a92a3a, but the filters were not.
That said, the respective count increased in Gentoo as well, so adjust
both input and output strings.  I'm updating Windows as a guesswork,
though I suspect that filter may not be necessary anymore, given that
CI was passing.
@charliermarsh
Copy link
Member

Thank you so much, I'm sorry about that.

@charliermarsh charliermarsh added the testing Internal testing of behavior label May 14, 2024
@charliermarsh charliermarsh merged commit e64c337 into astral-sh:main May 14, 2024
44 checks passed
@mgorny mgorny deleted the dist_cached-test-inc branch May 14, 2024 18:27
@mgorny
Copy link
Contributor Author

mgorny commented May 14, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Internal testing of behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants