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

Remove jaxen dependency #62

Merged
merged 1 commit into from
Oct 18, 2022
Merged

Remove jaxen dependency #62

merged 1 commit into from
Oct 18, 2022

Conversation

cies
Copy link
Member

@cies cies commented Oct 18, 2022

RePlay (and Play1) merely provide a thin wrapper over the Jaxen library. Otherwise the dependency was not used by RePlay.

If someone misses this wrapper it is easy for them to copy the framework/src/play/libs/XPath.java file (removed by this PR) into their project while adding the jaxen dependency (removed by this PR) to their project as well.

@cies cies added this to the 1.11.0 milestone Oct 18, 2022
@asolntsev asolntsev self-requested a review October 18, 2022 19:51
@asolntsev asolntsev self-assigned this Oct 18, 2022
@asolntsev asolntsev merged commit c13f986 into master Oct 18, 2022
@asolntsev asolntsev deleted the remove-jaxen-dependency branch October 18, 2022 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants