Claude Warren
Claude Warren
Some of the items I listed can be fixed by adding headers, others I think can be addressed by adding README or other documentation in the directory tree where the...
Except where noted all changes requested by @aherbert have been made.
Sorry been buried. I'll try to get back to this shortly.
the Java 20 failure reads: ``` org.apache.commons.collections4.IteratorUtilsTest.testGetIterator(IteratorUtilsTest.java:781) ``` So it appears to be in a different package. I think this package can be merged. Any objections from @garydgregory or @aherbert
Build seems to fail in code not added by me
@ottlinger @jbonofre I spent some time looking at this and discovered that there were a few junit4 annotations still in use. The one I was most concerned about was the...
@ottlinger @jbonofre does anyone see any reason not to merge this now?
All in all I think it looks good. Once merged we will need to go though each included/generated document and make changes, but that is a different ticket.
@ottlinger Do you want to solve all the documentation issues as part of this ticket or should we open separate tickets for those?
@kathia-barahona can you mark this as draft until it is ready for review. Also, do you know why this was not merged in 2023 when it was originally proposed?