Skip to content

Conversation

@cliserkad
Copy link
Contributor

@cliserkad cliserkad commented Mar 24, 2025

Context

Removes log messages that are intended as human readable real time progress bars. Real time monitoring of logs is impractical as we test many versions/environment setups in parallel. This change is intended to make repo maintenance easier.

Contributor Checklist

  • [✅] Added relevant integration or unit tests to verify the changes
    (this alters log messages only, no actual behavior / test changes)
  • [✅] Update the Readme or any other documentation (including relevant Javadoc)
    (no relevant documentation, maybe add an external link inside of workflow?)
  • [❌] Ensured that tests pass locally: mvn clean package
    (git signing config issue)
  • [✅] Ensured that the code meets the current checkstyle coding style definition: mvn clean verify -Pcheckstyle -Dmaven.test.skip=true -B

@TheSnoozer
Copy link
Collaborator

I would doubt anyone tries to parse log messages, but sure let me get this integrated.

As reference I checked if this option is available in the currently minimal version (https://maven.apache.org/docs/3.6.3/release-notes.html) and it's also still available in the latest available (https://maven.apache.org/ref/4.0.0-beta-3/maven-embedder/cli.html)

@TheSnoozer TheSnoozer merged commit fd70ec2 into git-commit-id:master Mar 25, 2025
22 checks passed
@TheSnoozer TheSnoozer added this to the next milestone Mar 25, 2025
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.

2 participants