Skip to content
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

Ability to override verbose from command line #341

Merged
merged 2 commits into from
Sep 2, 2024

Conversation

Pfoerd
Copy link
Contributor

@Pfoerd Pfoerd commented Jul 3, 2023

When analyzing configuration failures of Pitest it's often necessary to temporarily turn on verbose output. Changing the configuration in gradle files is possible but obviously has some drawbacks.

This change adds the ability to turn on verbose output from the command line via an optional parameter:

./gradlew pitest --verbose

Fixes #349

@Pfoerd
Copy link
Contributor Author

Pfoerd commented Jul 7, 2023

@szpak what do you think about this addition?

on-behalf-of: @e-solutions-GmbH <info@esolutions.de>
@Pfoerd
Copy link
Contributor Author

Pfoerd commented Sep 2, 2024

Hi @szpak, any chance to get this released in the near future? We still have a great need for this :)

@szpak
Copy link
Owner

szpak commented Sep 2, 2024

A good question. I don't remember, why I didn't merge it back then 🤔

@szpak szpak changed the base branch from master to devel September 2, 2024 19:00
@szpak szpak merged commit 1772a5e into szpak:devel Sep 2, 2024
2 checks passed
@szpak
Copy link
Owner

szpak commented Sep 2, 2024

I've merged it into devel and it will be included in the next release. I have to do some housekeeping work before and in addition, due to the "recent" changes in Sonatype the releasing from CI might not work anymore. In the end in might take a moment to release.

As a workaround, you might define something like:

pitest {
    verbose = project.findProperty("workaroundPitestVerbose") /*+ coercing string into boolean*/ ?: "false"
...

in your build.gradle to be able to control it from the command line (probably you already have it defined that way ;-) ).

Thanks for your PR. Sorry for delay (I might missed it (?), as it looks good and safe 🤔 ).

@Pfoerd
Copy link
Contributor Author

Pfoerd commented Sep 5, 2024

No worries and thank you! You may also consider #350 for the next releases ;)

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.

Ability to override verbose from command line
2 participants