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

Xpathfactory for matchers and assertions #132

Merged
merged 2 commits into from Aug 11, 2018

Conversation

Projects
None yet
2 participants
@bodewig
Member

bodewig commented Aug 10, 2018

This makes the XpathFactory used by XPath matchers/assertions configurable, for the rationale see #131

@krystiankaluzny could you please check whether the way I added it to the assertions is not too far from being idiomatic?

@bodewig bodewig requested a review from krystiankaluzny Aug 10, 2018

@bodewig bodewig merged commit e060b2c into master Aug 11, 2018

3 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls First build on xpathfactory-for-matchers-and-assertions at 91.795%
Details

@bodewig bodewig deleted the xpathfactory-for-matchers-and-assertions branch Aug 11, 2018

bodewig added a commit that referenced this pull request Aug 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment