gnodet commented on code in PR #13059:
URL: https://github.com/apache/maven/pull/13059#discussion_r4045025963
##########
impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/mvnup/goals/PluginUpgradeStrategy.java:
##########
@@ -939,10 +1021,11 @@ private PluginAnalysis analyzePluginsFromEffectiveModel(
needsDirectOverride.add(pluginKey);
context.debug("Plugin " + pluginKey + " version "
+ effectiveVersion
+ " has explicit version in inherited
build/plugins"
- + " — needs direct override to " +
upgrade.minVersion());
+ + " — needs direct version override to " +
upgrade.minVersion());
Review Comment:
Yes, it's intentional — em dashes (`—`) are used consistently as a style
choice in debug messages throughout this file (13 occurrences). This one was
already present in the pre-PR codebase; this PR only renamed `direct override
to` → `direct version override to` for clarity.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
