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

Compressed option doesn't work when using --junit #596

Open
philippb opened this issue Dec 17, 2024 · 1 comment
Open

Compressed option doesn't work when using --junit #596

philippb opened this issue Dec 17, 2024 · 1 comment
Labels
documentation Improvements or additions to documentation

Comments

@philippb
Copy link

Expected

When I use -c or --compress then it will compress the XML of --junit as well

Actual

-c only works without --junit.

The docs don't mention anything about this

Compare file output names for the two calls.

philipp@Philipps-CU-Mac ClickUp20_readme % vacuum report -c --junit openapi.yaml test


██╗   ██╗ █████╗  ██████╗██╗   ██╗██╗   ██╗███╗   ███╗
██║   ██║██╔══██╗██╔════╝██║   ██║██║   ██║████╗ ████║
██║   ██║███████║██║     ██║   ██║██║   ██║██╔████╔██║
╚██╗ ██╔╝██╔══██║██║     ██║   ██║██║   ██║██║╚██╔╝██║
 ╚████╔╝ ██║  ██║╚██████╗╚██████╔╝╚██████╔╝██║ ╚═╝ ██║
  ╚═══╝  ╚═╝  ╚═╝ ╚═════╝ ╚═════╝  ╚═════╝ ╚═╝     ╚═╝


version: 0.14.3 | compiled: Tue, 19 Nov 2024 17:42:39 UTC
🔗 https://quobix.com/vacuum | https://github.com/daveshanley/vacuum


 INFO  Linting against 46 rules: https://quobix.com/vacuum/rulesets/recommended
 SUCCESS  JUnit Report generated for 'openapi.yaml', written to 'test-12-16-24-22_58_41.xml'

using compress without junit

philipp@Philipps-CU-Mac ClickUp20_readme % vacuum report -c openapi.yaml test 


██╗   ██╗ █████╗  ██████╗██╗   ██╗██╗   ██╗███╗   ███╗
██║   ██║██╔══██╗██╔════╝██║   ██║██║   ██║████╗ ████║
██║   ██║███████║██║     ██║   ██║██║   ██║██╔████╔██║
╚██╗ ██╔╝██╔══██║██║     ██║   ██║██║   ██║██║╚██╔╝██║
 ╚████╔╝ ██║  ██║╚██████╗╚██████╔╝╚██████╔╝██║ ╚═╝ ██║
  ╚═══╝  ╚═╝  ╚═╝ ╚═════╝ ╚═════╝  ╚═════╝ ╚═╝     ╚═╝


version: 0.14.3 | compiled: Tue, 19 Nov 2024 17:42:39 UTC
🔗 https://quobix.com/vacuum | https://github.com/daveshanley/vacuum


 INFO  Linting against 46 rules: https://quobix.com/vacuum/rulesets/recommended
 SUCCESS  Report generated for 'openapi.yaml', written to 'test-12-16-24-22_59_25.json.gz'
@daveshanley
Copy link
Owner

This is by design, but you're right the docs need updating!

@daveshanley daveshanley added the documentation Improvements or additions to documentation label Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants