jdaugherty commented on PR #2245: URL: https://github.com/apache/groovy/pull/2245#issuecomment-2931453412
I've made the requested change to only sort if there's > 1 entry in the list. I also compiled with 4.0.27 and then with this change for the grails-scaffolding project. Here's one example of the GenerateAsyncControllerCommand.class and how this change affects it: * https://gist.github.com/jdaugherty/e0f560faed06a75bcda12f01bda3aa14 - 4.0.27 * https://gist.github.com/jdaugherty/ef60dccc4682ab588b01e8d7d6761c48 - Proposed Change I've also additionally run the reproducible build test script in grails-core and confirmed the differences drop off after this change. @eric-milles if you have suggestions on how I could create an automated test for this, I'm happy to pursue this too. I'm hoping the testing I've done on the grails-core project would suffice though. -- 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: notifications-unsubscr...@groovy.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
