Skip to content

Update documentation for vuln scanning to match behaviour#8

Merged
andrew merged 1 commit intogit-pkgs:mainfrom
ashokfaire:tweak-vulns-scan-example-in-readme
Feb 25, 2026
Merged

Update documentation for vuln scanning to match behaviour#8
andrew merged 1 commit intogit-pkgs:mainfrom
ashokfaire:tweak-vulns-scan-example-in-readme

Conversation

@ashokfaire
Copy link

@ashokfaire ashokfaire commented Feb 25, 2026

  • Adjust the examples for running a vulnerability scan. (179c679)

    To run the scan, I believe you need to pass the subcommand. It looks like these documentation examples arrived very early in the repo and haven't changed, so I assume the behaviour I'm getting in 0.14.0 should match the documentation.


If I run the first example in the index page, git pkgs vulns in 0.14.0 I get back the help message for vulns' subcommands.

(The documentation here is still somewhat incomplete, but I've filed a separate issue about that: git-pkgs/git-pkgs#131.)

@ashokfaire ashokfaire changed the title Adjust the examples for running a vulnerability scan Update documentation for vuln scanning to match behaviour Feb 25, 2026
@andrew
Copy link
Contributor

andrew commented Feb 25, 2026

Good catch, thanks!

@andrew
Copy link
Contributor

andrew commented Feb 25, 2026

The build failure isn't your fault, I'm working on a fix seperately

To run the scan, I believe you need to pass the subcommand. It looks like these documentation examples arrived very early in the repo and haven't changed, so I assume the behaviour I'm getting in 0.14.0 should match the documentation.
@andrew andrew force-pushed the tweak-vulns-scan-example-in-readme branch from 179c679 to 3bfa27e Compare February 25, 2026 15:08
@andrew andrew merged commit 75b9c46 into git-pkgs:main Feb 25, 2026
1 check passed
@ashokfaire ashokfaire deleted the tweak-vulns-scan-example-in-readme branch February 25, 2026 15:15
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.

2 participants