Claude Warren

Results 52 comments of 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?