[
https://issues.apache.org/jira/browse/MNG-7443?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17560651#comment-17560651
]
ASF GitHub Bot commented on MNG-7443:
-------------------------------------
Giovds commented on code in PR #701:
URL: https://github.com/apache/maven/pull/701#discussion_r910382076
##########
maven-core/src/main/java/org/apache/maven/DefaultMaven.java:
##########
@@ -132,6 +137,7 @@ public DefaultMaven(
this.buildResumptionAnalyzer = buildResumptionAnalyzer;
this.buildResumptionDataRepository = buildResumptionDataRepository;
this.superPomProvider = superPomProvider;
+ this.projectSelector = new ProjectSelector();
Review Comment:
The reasoning behind this is that the `ProjectSelector` is likely only going
to be used in two classes (`DefaultMaven` and `DefaultGraphBuilder`) and does
not require any complex configuration to be created. It's relatively
lightweight and I would have liked to make it package-private as well but since
`DefaultMaven` is in a different package this wasn't possible. If it preferred
to subject it to DI I'll happily change it.
> Consistent logging between optional projects and optional profiles
> ------------------------------------------------------------------
>
> Key: MNG-7443
> URL: https://issues.apache.org/jira/browse/MNG-7443
> Project: Maven
> Issue Type: Improvement
> Components: Core, Logging
> Affects Versions: 4.0.0-alpha-1
> Reporter: Giovanni van der Schelde
> Priority: Minor
> Attachments: example.png
>
>
> Maven 4 introduces optional profiles and optional projects. However, the
> feedback provided to the user on whether a project or profile has been
> skipped is inconsistent between the two (see image attached).
> For profiles, it will be logged twice: before and after the build.
> For projects, it will be logged once: before the build.
> The idea would be to log the information for skipped optional projects after
> the build as well.
> !example.png!
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
