Skip to content

Fix: wire up --verbose flag to gate PASS output in validate.py#898

Open
jcarr710 wants to merge 2 commits into
Prowlarr:masterfrom
jcarr710:master
Open

Fix: wire up --verbose flag to gate PASS output in validate.py#898
jcarr710 wants to merge 2 commits into
Prowlarr:masterfrom
jcarr710:master

Conversation

@jcarr710

Copy link
Copy Markdown

The --verbose flag was defined in argparse but never passed through
to validate_directory or validate_files_in_directory, so it had
no effect regardless of whether it was passed or not.

This fix wires verbose through the call chain so that PASS output
is only shown when --verbose is passed. Default behavior now shows
only failures and the summary, which is more useful when validating
500+ files.

Before: python scripts/validate.py always printed all PASS lines
After: python scripts/validate.py shows summary only
python scripts/validate.py --verbose shows all PASS lines

@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant