-
Notifications
You must be signed in to change notification settings - Fork 145
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
Implemented DrainFilter #292
Conversation
I've reviewed this now, and I'm now doubly sure it does what it should. Review by another human would be appreciated, but from my side, I think it's good to go. |
Anything I still need to do with this? |
Sorry for the delay. I'll try to review this soon (if another maintainer doesn't get to it first). |
No worries! I just wanted to confirm if I still need to do something before the review process can start. |
☔ The latest upstream changes (presumably #298) made this pull request unmergeable. Please resolve the merge conflicts. |
@bors-servo r+ Sorry for the wait! |
📌 Commit ae7ff95 has been approved by |
☀️ Test successful - checks-github |
Provides an implementation for DrainFilter (issue #281).