Skip to content

Allow @...@ to match none or more elements #97

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Jul 20, 2017
Merged

Conversation

Gamesh
Copy link
Contributor

@Gamesh Gamesh commented Apr 10, 2017

fixes issue #72, by allowing @...@ to match none or more elements. Also added unit tests

@Gamesh Gamesh changed the title master Allow @...@ to match none or more elements Apr 19, 2017
@norberttech
Copy link
Member

could you also please add an example of this feature into https://github.com/coduo/php-matcher/blob/master/tests/MatcherTest.php if possible maybe in test_matcher_with_json

@Gamesh
Copy link
Contributor Author

Gamesh commented Apr 20, 2017

@norzechowicz done

@norberttech
Copy link
Member

@Gamesh sorry it took me that long : / I added that to my todo list and forgot to merge it later

@norberttech norberttech merged commit 85e6c89 into coduo:master Jul 20, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants