Hello Marco, Good call coming to the mailing list first :-)
There is a mismatch in concepts in the proposed API IMO. But I can't tell for sure since you don't describe the method: there are no input, output, and edge case descriptions. I would propose the method with what the Javadoc would look like for example. Either you want to add a method to ListUtils and pass in a List..., or, you want to add a method to CollectionUtils and pass in a Collection..., or you want to add a method to IterableUtils and pass in an Iterable... The current set of APIs is not perfectly organized though. Is it more than return Stream.of(arrayOfLists).filter(List::isEmpty).findFirst() ? HTH, Gary On Wed, Mar 20, 2024 at 7:30 AM Marco Dell'Anna <marco.della...@gmail.com> wrote: > > Hi, > I would like to have a CollectionUtils.firstNotEmpty(List... lists) method. > Since commons-collections/CONTRIBUTING.md at master · > apache/commons-collections · GitHub > <https://github.com/apache/commons-collections/blob/master/CONTRIBUTING.md> > says: > * If you're planning to implement a new feature it makes sense to discuss > your changes on the dev list first. This way you can make sure you're not > wasting your time on something that isn't considered to be in Apache > Commons Collections's scope. > I'm asking if I should create a Merge Request > > Thanks > Marco Dell'Anna --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
