Skip to content

MNG-7438 add execution id to "configuring mojo" debug message#700

Merged
hboutemy merged 1 commit intomaven-3.9.xfrom
MNG-7438
Apr 15, 2022
Merged

MNG-7438 add execution id to "configuring mojo" debug message#700
hboutemy merged 1 commit intomaven-3.9.xfrom
MNG-7438

Conversation

@hboutemy
Copy link
Copy Markdown
Member

@hboutemy hboutemy commented Mar 22, 2022

[DEBUG] Loading mojo org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test from plugin realm ClassRealm[plugin>org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5, parent: sun.misc.Launcher$AppClassLoader@7852e922]
[DEBUG] Configuring mojo execution 'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test:default-test' with basic configurator -->

@hboutemy hboutemy changed the title MNG-7438 MNG-7438 add execution id to "configuring mojo" debug message Mar 26, 2022
@hboutemy hboutemy requested a review from michael-o March 27, 2022 09:04
[DEBUG] Configuring mojo execution
'org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M5:test:default-test'
with basic configurator

rephrase "Configuring" to "Loading" mojo "from plugin realm"
to differentiate from "configuring mojo" = apply configuration

rename populatePluginFields(..) private method
to populateMojoExecutionFields(..) to better match the intent
@hboutemy hboutemy merged commit 3c6009c into maven-3.9.x Apr 15, 2022
@hboutemy hboutemy deleted the MNG-7438 branch April 15, 2022 16:42
@jira-importer
Copy link
Copy Markdown

Resolve #9289

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants