gnodet-bot commented on code in PR #12694:
URL: https://github.com/apache/maven/pull/12694#discussion_r4046222244
##########
api/maven-api-core/src/main/java/org/apache/maven/api/plugin/Log.java:
##########
@@ -87,8 +87,6 @@ default void trace(Throwable error) {}
* The supplier is only evaluated if trace is enabled.
* <p>
* The default implementation is a no-op for backward compatibility.
- *
- * @param content the message supplier
*/
default void trace(Supplier<String> content) {}
Review Comment:
**[medium] `@param content` tag dropped — regression from `caeeeb4`**
This `@param` tag was explicitly added in commit `caeeeb4` (confirmed in the
APPROVE of `63ed04e`/`caeeeb4`). The squash-rebase dropped it. Every other
supplier-based method in this interface documents its `@param content`.
```suggestion
* The default implementation is a no-op for backward compatibility.
*
* @param content the message supplier
*/
default void trace(Supplier<String> content) {}
```
--
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]
