Copilot commented on code in PR #2324: URL: https://github.com/apache/groovy/pull/2324#discussion_r2466405275
########## src/main/java/org/apache/groovy/parser/antlr4/AstBuilder.java: ########## @@ -151,16 +155,222 @@ import java.util.stream.Collectors; import static groovy.lang.Tuple.tuple; -import static org.apache.groovy.parser.antlr4.GroovyParser.*; +import static org.apache.groovy.parser.antlr4.GroovyParser.ADD; +import static org.apache.groovy.parser.antlr4.GroovyParser.ARROW; +import static org.apache.groovy.parser.antlr4.GroovyParser.AS; +import static org.apache.groovy.parser.antlr4.GroovyParser.ASYNC; Review Comment: [nitpick] The wildcard import at line 157 has been replaced with explicit imports. While this improves clarity by showing exactly which symbols are used, it significantly increases the number of import statements (158-362). Consider whether this trade-off between explicitness and verbosity aligns with the project's style guidelines. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
