[
https://issues.apache.org/jira/browse/MNG-7098?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17602005#comment-17602005
]
ASF GitHub Bot commented on MNG-7098:
-------------------------------------
Giovds commented on code in PR #783:
URL: https://github.com/apache/maven/pull/783#discussion_r966396162
##########
maven-embedder/src/main/java/org/apache/maven/cli/event/ExecutionEventLogger.java:
##########
@@ -143,7 +143,11 @@ public void sessionStarted( ExecutionEvent event )
project.getName(), chars( ' ', ( len > 0 ) ? len : 1
), project.getPackaging() );
}
- totalProjects = projects.size();
+ final List<MavenProject> allProjects =
event.getSession().getAllProjects();
Review Comment:
Perhaps it might be worth noting that the `MavenSession#getAllProjects()`
[states](https://github.com/apache/maven/blob/3c487fa697c3ff29d6c79ab5cf6e834abfd143b3/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java#L273)
`This is a provisional method and may be removed`. However long that may have
been there :see_no_evil:.
> Project counter should be cumulative when using resume
> ------------------------------------------------------
>
> Key: MNG-7098
> URL: https://issues.apache.org/jira/browse/MNG-7098
> Project: Maven
> Issue Type: Improvement
> Components: Reactor and Workspace
> Reporter: Robert Scholte
> Priority: Minor
> Fix For: 4.0.x-candidate
>
>
> When doing a resume on a build, the counter now starts again at 1.
> E.g. Suppose a project has 10 modules and the 7th module fails. With a resume
> now you'll see {{[1/4]}}.
> To me it makes more sense to say {{[7/10]}}. You're still as close to the end
> as usual, but this indicates the size of the complete project.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
