desruisseaux commented on code in PR #11857:
URL: https://github.com/apache/maven/pull/11857#discussion_r3292589443
##########
impl/maven-impl/src/main/java/org/apache/maven/impl/PathModularizationCache.java:
##########
@@ -189,10 +189,12 @@ Optional<String>
warningForFilenameBasedAutomodules(Collection<Path> modulePaths
return Optional.empty();
}
String lineSeparator = System.lineSeparator();
+ String fileSeparator = lineSeparator + " - ";
var joiner = new StringJoiner(
- lineSeparator + " - ",
- "Filename-based automodules detected on the module path: " +
lineSeparator + " - ",
- lineSeparator + "Please don't publish this project to a public
artifact repository.");
+ fileSeparator,
+ "Filename-based automodules detected on the module path: " +
fileSeparator,
+ lineSeparator + "This project may not work in applications
that do not use "
+ + "these dependencies with exactly the same filenames
as listed above.");
automodulesDetected.forEach(joiner::add);
Review Comment:
I clicked on "commit suggestion" too quickly. The replacement proposed by
Claude is good, but it is replacing the wrong lines. I will fix and do a forced
push.
--
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]
