search

pyupio / safety

Safety checks Python dependencies for known security vulnerabilities and suggests the proper remediations for vulnerabilities detected.
https://safetycli.com/product/safety-cli
MIT License
1.7k stars 143 forks source link

`--bare` and `--full-report` should be mutually exclusive #346

Closed sobolevn closed 2 years ago

sobolevn commented 3 years ago

I ran into this problem way too often 🙂

Like here: https://github.com/wemake-services/docker-image-size-limit/blob/4b9a9c690d6b4ce2d90419ac9bed13f6898659af/.github/workflows/test.yml#L48

I propose to make --bare and --full-report CLI options for check command mutually exclusive. So, passing safety check --bare --full-report would result in ArgumentError. Right now it fallbacks to --bare.

yeisonvargasf commented 2 years ago

Thanks for the report @sobolevn; Safety >= 2.0.0 fixes this issue.