-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MNG-8222] Extend PropertyContributor #1679
base: master
Are you sure you want to change the base?
Conversation
Add more contextual info to it. --- https://issues.apache.org/jira/browse/MNG-8222
api/maven-api-spi/src/main/java/org/apache/maven/api/spi/PropertyContributor.java
Outdated
Show resolved
Hide resolved
for (PropertyContributor contributor : effectivePropertyContributors.values()) { | ||
contributor.contribute(userPropertiesMap); | ||
Map<String, String> contribution = contributor.contribute(protoSession); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A given contributor won't see properties added by previous contributors, is that what we want ?
@Experimental | ||
@ThreadSafe | ||
public interface ProtoSession { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should give access to CLI commands / arguments I think, this may be needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe just a List<String>
to not expose commons-cli.
We will rework this, but leaving as DRAFT for reference |
Add more contextual info to it.
https://issues.apache.org/jira/browse/MNG-8222