Copilot commented on code in PR #2228:
URL: https://github.com/apache/groovy/pull/2228#discussion_r2096073259
##########
subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/GroovyDocTool.java:
##########
@@ -45,11 +50,12 @@ public GroovyDocTool(String[] sourcepaths) {
}
public GroovyDocTool(ResourceManager resourceManager, String[]
sourcepaths, String classTemplate) {
- this(resourceManager, sourcepaths, new String[]{}, new String[]{}, new
String[]{classTemplate}, new ArrayList<LinkArgument>(), new Properties());
+ this(resourceManager, sourcepaths, new String[]{}, new String[]{}, new
String[]{classTemplate}, new ArrayList<LinkArgument>(), null, new Properties());
}
Review Comment:
This constructor introduces a new `javaVersion` parameter but lacks JavaDoc
describing its purpose, accepted format, and behavior. Please add a doc comment.
```suggestion
/**
* Constructs a GroovyDocTool instance with the specified parameters.
*
* @param resourceManager the resource manager for handling resources,
or null if not required
* @param sourcepaths the paths to the source files to be processed
* @param docTemplates the templates for generating documentation
* @param packageTemplates the templates for generating package-level
documentation
* @param classTemplates the templates for generating class-level
documentation
* @param links a list of link arguments for external
references
* @param javaVersion the Java version to be used for parsing and
processing (e.g., "1.8", "11", "17");
* determines the Java language level
* @param properties additional properties for configuration
*/
```
--
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
