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: filtering by regex #1089
fix: filtering by regex #1089
Conversation
Thanks for the PR! I assume you are working on figuring it out a fix? |
Hey @gr2m I think this should look good now. I also added test, please review. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We try to do our best, but nock is maintained by volunteers and there is only so much we can do at a time. Thank you for your contributions. |
@hisabimbola It looks like we haven't reviewed this again. I'm reopening it. |
@gr2m I think this is ready; what do you think? |
Let me know if any changes are needed aside resolving the conflict |
This looks fine to me! @gr2m Any time to check this again? |
yep looks good, just need rebasing :) Could you do that @hisabimbola? |
yeah @gr2m updated the PR |
Thanks Abimbola! |
🎉 This PR is included in version 10.0.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue and add a reference to this one if it’s related. Thank you! |
Add failing tests for #1080