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: filtering by regex #1089

Merged
merged 5 commits into from Nov 26, 2018
Merged

fix: filtering by regex #1089

merged 5 commits into from Nov 26, 2018

Conversation

hisabimbola
Copy link
Contributor

Add failing tests for #1080

@hisabimbola hisabimbola changed the title test: fix filtering by regex WIP fix: filtering by regex Mar 4, 2018
@gr2m
Copy link
Member

gr2m commented Mar 4, 2018

Thanks for the PR! I assume you are working on figuring it out a fix?

@hisabimbola hisabimbola changed the title WIP fix: filtering by regex fix: filtering by regex Mar 18, 2018
@hisabimbola
Copy link
Contributor Author

Hey @gr2m

I think this should look good now. I also added test, please review.

@stale
Copy link

stale bot commented Sep 13, 2018

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.

@RichardLitt
Copy link
Member

@hisabimbola It looks like we haven't reviewed this again. I'm reopening it.

@RichardLitt RichardLitt reopened this Sep 20, 2018
@stale stale bot removed the stale label Sep 20, 2018
@RichardLitt
Copy link
Member

@gr2m I think this is ready; what do you think?

@hisabimbola
Copy link
Contributor Author

Let me know if any changes are needed aside resolving the conflict

@RichardLitt
Copy link
Member

This looks fine to me! @gr2m Any time to check this again?

@gr2m
Copy link
Member

gr2m commented Nov 13, 2018

yep looks good, just need rebasing :) Could you do that @hisabimbola?

@hisabimbola
Copy link
Contributor Author

yeah @gr2m updated the PR

@gr2m gr2m merged commit 88fbdc2 into nock:master Nov 26, 2018
@gr2m
Copy link
Member

gr2m commented Nov 26, 2018

Thanks Abimbola!

@nockbot
Copy link
Collaborator

nockbot commented Dec 3, 2018

🎉 This PR is included in version 10.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lock
Copy link

lock bot commented Dec 17, 2018

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!

@lock lock bot locked as resolved and limited conversation to collaborators Dec 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants