Skip to content

add --no-transfer-progress option to mvn commands #838

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 25, 2025

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