gnodet commented on PR #1086:
URL: https://github.com/apache/maven/pull/1086#issuecomment-1508057578
The proposed syntax is problematic to me as it's not coherent with the one
displayed to the user.
Users now see:
```
[INFO]
[INFO] --- checkstyle:3.0.0:check (checkstyle-check) @ core-it-suite ---
[INFO]
[INFO] --- enforcer:3.1.0:enforce (enforce-bytecode-version) @ core-it-suite
---
[INFO] Adding ignore: module-info
[INFO]
[INFO] --- enforcer:3.1.0:enforce (enforce-maven-version) @ core-it-suite ---
[INFO]
[INFO] --- enforcer:3.1.0:enforce (enforce-java-version) @ core-it-suite ---
[INFO]
[INFO] --- remote-resources:1.7.0:process (process-resource-bundles) @
core-it-suite ---
[WARN] The POM for
org.apache.maven.wagon:wagon-http-lightweight:jar:1.0-beta-6 is invalid,
transitive dependencies (if any) will not be available, enable verbose output
(-X) for more details
```
so we need to use a syntax which actually supports what is displayed such as
`enforcer:3.1.0:enforce`.
So it should be coherent with:
*
https://github.com/apache/maven/blob/master/maven-core/src/main/java/org/apache/maven/lifecycle/internal/DefaultLifecycleTaskSegmentCalculator.java#L95
*
https://github.com/apache/maven/blob/master/maven-embedder/src/main/java/org/apache/maven/cli/event/ExecutionEventLogger.java#L422-L431
--
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]
