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

Make call to beanutils.ConvertUtils optional (fixes JXPATH-190) #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TobiX
Copy link

@TobiX TobiX commented Mar 15, 2017

@TobiX
Copy link
Author

TobiX commented Mar 16, 2017

Test failure is not releated to this pull request, a quick bisect shows its broken quite a long time... org.apache.commons.jxpath.ri.model.dynabeans.LazyDynaBeanTest#testLazyProperty should probably not be named "testLazyProperty", since it's not the main JUnit test.

@markt-asf markt-asf closed this May 1, 2019
@TobiX
Copy link
Author

TobiX commented May 1, 2019

Why was this closed?

@markt-asf
Copy link

The PR was against trunk
The trunk branch was deleted as all Commons projects use maser in preference to trunk. That automatically closed all PRs against trunk.
I'll re-open this but you'll need to modify the PR to be against master rather than trunk.

@markt-asf markt-asf reopened this May 1, 2019
@TobiX TobiX changed the base branch from trunk to master May 1, 2019 22:40
@TobiX TobiX force-pushed the fix-JXPATH-190 branch from ac85ee2 to 3fff47b Compare May 1, 2019 22:45
@TobiX TobiX force-pushed the fix-JXPATH-190 branch from 0e8dc36 to c36d377 Compare May 3, 2021 09:07
@TobiX TobiX force-pushed the fix-JXPATH-190 branch 3 times, most recently from d9abb6d to 3b0f302 Compare October 16, 2023 07:57
@TobiX
Copy link
Author

TobiX commented Oct 16, 2023

Ping @garydgregory - Since you have worked on this project recently, would you be willing to review and merge this?

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