(#1560) Delegate `TextOf(Iterator)` to `TextOf(Iterable)`

yegor256 / cactoos

Object-Oriented Java primitives, as an alternative to Google Guava and Apache Commons

https://www.cactoos.org

MIT License

737 stars 163 forks source link

(#1560) Delegate `TextOf(Iterator)` to `TextOf(Iterable)` #1599

Closed DmitryBarskov closed 3 years ago

DmitryBarskov commented 3 years ago

For #1560:

Constructor TextOf(Iterator) now delegates to TextOf(Iterable)
Constructor TextOf parameter is typed as Iterable<Character> instead of Iterable<?>
Add extra test for TextOf(Iterable)
Replace usages of TextOf(Iterable<?>) with Joined and Concatenated

victornoel commented 3 years ago

@DmitryBarskov @andreoss thx

victornoel commented 3 years ago

@rultor merge

rultor commented 3 years ago

@rultor merge

@victornoel OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 3 years ago

@rultor merge

@victornoel Done! FYI, the full log is here (took me 13min)

0crat commented 3 years ago

@sereshqua/z please review this job completed by @andreoss/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

sereshqua commented 3 years ago

@DmitryBarskov please make sure you start all of the comments with the name of the user they are referred to, see

sereshqua commented 3 years ago

@0crat quality acceptable