arturobernalg commented on PR #13727: URL: https://github.com/apache/maven/pull/13727#issuecomment-6518487914
@gnodet agreed -- I don't see a use case either, and this PR doesn't add one. The only thing exported here is the single `Ansi` class maven-jline already ships (for maven-shared-utils's `AnsiMessageBuilder` compat), narrowed from the old wildcard; `AnsiConsole` was a separate suggestion from @slawekjaranowski that I looked at and intentionally left out for exactly the reason you're raising (no real implementation behind it, and it would let a plugin call `systemInstall()`/`systemUninstall()` and hijack the shared stdout/stderr). Happy to keep it out unless/until there's an actual forwarder to back it. So as far as I can tell this PR is unaffected by that discussion -- just flagging in case you want to review the diff as-is. -- 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]
