Skip to content

Clarify how late placeholder replacement (@{...}) deals with#3208

Merged
kwin merged 1 commit intomasterfrom
doc/clarify-late-replacement-in-argline
Oct 27, 2025
Merged

Clarify how late placeholder replacement (@{...}) deals with#3208
kwin merged 1 commit intomasterfrom
doc/clarify-late-replacement-in-argline

Conversation

@kwin
Copy link
Member

@kwin kwin commented Oct 23, 2025

unresolvable parameters

Also outline that this syntax is only necessary for property values which are modified, not for those which are exclusively set through other plugins due to how resolving works for plugin parameters (org.apache.maven.plugin.PluginParameterExpressionEvaluator)

Following this checklist to help us incorporate your
contribution quickly and easily:

  • Each commit in the pull request should have a meaningful subject line and body.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Run mvn clean install to make sure basic checks pass. A more thorough check will
    be performed on your pull request automatically.
  • You have run the integration tests successfully (mvn -Prun-its clean install).

If your pull request is about ~20 lines of code you don't need to sign an
Individual Contributor License Agreement if you are unsure
please ask on the developers list.

To make clear that you license your contribution under
the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

@kwin kwin force-pushed the doc/clarify-late-replacement-in-argline branch 2 times, most recently from f02e693 to 544eb47 Compare October 23, 2025 09:31
* <br>
* Since the Version 2.17 using an alternate syntax for {@code argLine}, <b>@{...}</b> allows late replacement
* of properties when the plugin is executed, so properties that have been modified by other plugins will be picked
* Since version 2.17 using an alternate syntax for {@code argLine}, <code>@{...}</code> allows late replacement
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can remove since .... it refers to very old one

unresolvable parameters

Also outline that this syntax is only necessary for property values
which are modified, not for those which are exclusively set through
other plugins due to how resolving works for plugin parameters
(org.apache.maven.plugin.PluginParameterExpressionEvaluator)
@kwin kwin force-pushed the doc/clarify-late-replacement-in-argline branch from 544eb47 to 3881c7d Compare October 27, 2025 10:13
@kwin kwin merged commit cfe85b9 into master Oct 27, 2025
39 checks passed
@kwin kwin deleted the doc/clarify-late-replacement-in-argline branch October 27, 2025 13:58
@kwin kwin added the documentation Improvements or additions to documentation label Oct 27, 2025
@github-actions github-actions bot added this to the 3.5.5 milestone Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants