-
Notifications
You must be signed in to change notification settings - Fork 5
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
feat(scan): CLI for PDI scanner #4018
Conversation
4a14dc4
to
2fe303b
Compare
2fe303b
to
547d6c2
Compare
@eventualbuddha I've been slacking here. Is a review still useful? |
Reviewing the TUI from a user perspective would be potentially useful, otherwise you can just let @kshen0 review when the VxScan integration is done. |
45ca2c7
to
80dbe32
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just from a TUI user perspective, as per @eventualbuddha 's request – I love having this and it looks powerful enough to test all the thing. 👍
Closing this out as we now have a proper integration and this CLI is now out of sync with main |
Overview
Requires getting the kernel module and
libPDIScan
shared library set up properly first. Needs a decent amount of cleanup and testing before we can merge this.To try it out,
cd libs/pdi-rs && cargo run --release
. More comprehensive instructions here.Demo Video or Screenshot
Demo video where I start an auto-scan session and scan a single document:
pdi-cli-review.mov
Testing Plan