sbglasius commented on code in PR #52:
URL: https://github.com/apache/groovy-website/pull/52#discussion_r1741550808
##########
generator/src/main/groovy/generator/SiteGenerator.groovy:
##########
@@ -287,7 +287,7 @@ class SiteGenerator {
@CompileDynamic
private void renderBlogFeed(Map<String, Document> blogList, String
baseDir) {
- def sorted = blogList.sort { e -> e.value.revisionInfo.date }
+ def sorted = blogList.sort { e -> e.value.revisionInfo.date.reverse() }
Review Comment:
Instead of using reverse (which should be outside the closure), this can
easily be done using the other variant of sort:
```
def sorted = blogList.sort { a, b ->
b.value.revisionInfo.date <=> a.value.revisionInfo.date
}
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@groovy.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
