efegokdemir opened a new pull request, #13251: URL: https://github.com/apache/maven/pull/13251
## Summary `mvnup check` performs a dry run, but its output described planned changes as already applied. This makes the command's report misleading even though the POMs are not written. Fixes #13250 ## Changes - Track dry-run state in the upgrade context. - Use planned-action wording for plugin upgrades and modification summaries during `mvnup check`. - Preserve the existing applied wording for `mvnup apply`. - Add regression coverage for dry-run state and orchestrator output. ## Testing - `mvn -pl impl/maven-cli -am -DskipITs -Dtest=StrategyOrchestratorTest,UpgradeContextTest,CheckTest -Dsurefire.failIfNoSpecifiedTests=false test` ✅ - `mvn -pl impl/maven-cli -am -DskipITs -DskipTests verify` ✅ - `git diff --check` ✅ ## Notes The change is limited to reporting semantics; it does not alter the underlying upgrade operations or the check goal's read-only behavior. This contribution was prepared with Codex assistance under human supervision. The changed code, tests, and results were reviewed before submission. -- 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]
