Skip to content

Output of all validated files if verbose: true #312

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

Closed
tschm opened this issue Aug 27, 2023 · 4 comments · Fixed by #313
Closed

Output of all validated files if verbose: true #312

tschm opened this issue Aug 27, 2023 · 4 comments · Fixed by #313
Labels
enhancement New feature or request

Comments

@tschm
Copy link

tschm commented Aug 27, 2023

I would love to see in the output a list of those files that been picked up by your tool and validated.
Maybe it's already possible. I am a new user.

@sirosen sirosen added the enhancement New feature or request label Aug 27, 2023
@sirosen
Copy link
Member

sirosen commented Aug 27, 2023

That's a good idea! It's not available today but it makes perfect sense.

I'll need to do some work to make sure the JSON output also includes this info, so it's not quite as simple as just printing the files as they're checked, but it shouldn't be hard.

@sirosen
Copy link
Member

sirosen commented Aug 27, 2023

This should now be available in v0.26.2!

Please let me know if it's insufficient for your needs or if you run into any trouble with check-jsonschema.

@tschm
Copy link
Author

tschm commented Aug 28, 2023

It works. However, you may want to check whether you really need both the verbose: true and the args: ["--verbose"] construct... Many thanks for this

@sirosen
Copy link
Member

sirosen commented Aug 28, 2023

check-jsonschema is not responsible for the pre-commit verbosity option to which you are referring. Only two verbosity options are defined by this tool: -q/--quiet and -v/--verbose.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants