gnodet-bot commented on code in PR #13330:
URL: https://github.com/apache/maven/pull/13330#discussion_r4279035843
##########
api/maven-api-core/src/main/java/org/apache/maven/api/plugin/annotations/Mojo.java:
##########
@@ -76,8 +76,17 @@
boolean projectRequired() default true;
/**
- * if the Mojo uses the Maven project and its subprojects.
- * @return uses the Maven project and its subprojects
+ * Indicates whether the Mojo operates as an aggregator across the Maven
reactor and its subprojects.
+ * <p>
+ * <b>Execution semantics:</b>
+ * <ul>
+ * <li>When invoked directly via CLI (e.g. {@code mvn plugin:goal}), an
aggregator Mojo executes
+ * only once on the top-level project of the reactor (tasks are
executed in CLI argument order).</li>
Review Comment:
💡 **Nit:** The parenthetical `(tasks are executed in CLI argument order)` is
contextually awkward here — the bullet is about a single aggregator Mojo
executing once, but the parenthetical makes a general statement about all CLI
tasks. Consider removing it from this line and folding it into item 4 of
section 1.1 in `aggregator-goals.md` where task ordering is already documented
in detail.
```suggestion
* <li>When invoked directly via CLI (e.g. {@code mvn plugin:goal}),
an aggregator Mojo executes
* only once on the top-level project of the reactor.</li>
```
--
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]
