pzygielo commented on code in PR #320: URL: https://github.com/apache/maven-compiler-plugin/pull/320#discussion_r2042601266
########## src/it/MCOMPILER-366/verify.groovy: ########## @@ -19,6 +19,6 @@ buildLog = new File( basedir, 'build.log' ).text; -assert buildLog.contains("[WARNING] Filename-based automodules detected on the module-path:") +assert buildLog.contains("[WARNING] Filename-based automodules detected on the module path:") Review Comment: Why not keep `module-path` like `Filename-based` is? It's the option used by both - `javac` and `java`. Staying with `module-path` would eliminate few files from changeset at least. -- 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: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org