Removed more .streams() with our filterAndMap code by bbakerman · Pull Request #4044 · graphql-java/graphql-java
Conversation
Die .stream() die
We still have some left but they tend to be .findFirst() or flatMap and hence harder to replace in a straight forward manner
| .collect(Collectors.toList()); | ||
|
|
||
| List<String> names = ImmutableKit.map(directiveDefinition.getDirectiveLocations(), | ||
| it -> it.getName().toUpperCase()); |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The above is a double map - can be combined into one
Test Results
323 files 323 suites 2m 53s ⏱️
4 933 tests 4 923 ✅ 10 💤 0 ❌
5 022 runs 5 012 ✅ 10 💤 0 ❌
Results for commit b0e520b.
dondonz
deleted the
remove-more-streams
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
2 participants