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

Bugfix/issue 1451 #1472

Merged
merged 5 commits into from
Feb 7, 2021
Merged

Conversation

joelpramos
Copy link
Contributor

Description

Thanks for contributing this Pull Request. Make sure that you submit this Pull Request against the develop branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

@joelpramos
Copy link
Contributor Author

opening the fix for now, might add some more unit tests later for matches for contains, contains any etc

@ptrthomas
Copy link
Member

@joelpramos

image

I see the build failed. actually that test looks wrong ! so if you also agree, you can change the test

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.

2 participants