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

Add Sieve.deliver_multiple_users #4882

Merged
merged 1 commit into from
Apr 5, 2024
Merged

Conversation

ksmurchison
Copy link
Contributor

Test for #4878

@ksmurchison ksmurchison merged commit c431cda into master Apr 5, 2024
2 checks passed
@ksmurchison ksmurchison deleted the sieve_deliver_multiple_users branch April 5, 2024 14:08
@elliefm
Copy link
Contributor

elliefm commented Apr 8, 2024

I fiddled this test onto 3.6 (which doesn't have tiny tests, so I had to copy and paste the new test function into the main Sieve.pm) and it passes for 3.6. So whatever the bug in #4878 is, either this test doesn't reproduce it, or it was already fixed sometime between 3.6.1 and 3.6.4 (though I don't see anything in the commit log that looks relevant)

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

Successfully merging this pull request may close these issues.

3 participants