Uh oh!
There was an error while loading. Please reload this page.
docs(prover): correct prove() exit code doc comment - #2395
Conversation
All contributors have signed the DCO ✍️ ✅ |
eviehoward
commented
Jul 21, 2026
I have read the DCO document and I hereby sign the DCO. |
matthewgrossman
commented
Jul 21, 2026
recheck |
Signed-off-by: Evie Howard <evhoward@redhat.com> Signed-off-by: Evan Lezar <elezar@nvidia.com>
253f456 to
761f54aCompare
elezar
left a comment
There was a problem hiding this comment.
Thanks @eviehoward.
Your changes missed the fact that the function never returns 2 (at least in the current version) and returns the error code wrapped in a Result. This seems to have been incorrect from the start.
Instead of a review back and forth, I have addressed this in the branch.
elezar
commented
Jul 22, 2026
/ok-to-test 761f54a |
elezar
commented
Jul 22, 2026
Thanks for the documentation fix. While validating the
These are follow-up candidates only and do not block this PR. |
eviehoward
commented
Jul 22, 2026
I will take a look now, thank you! |
Uh oh!
There was an error while loading. Please reload this page.
Summary
Corrects the
prove()doc comment which incorrectly described exit code 1 asfiring on "critical or high findings." The prover has no severity grades — exit
code 1 fires on any unaccepted finding.
Related Issue
Closes#2394
Changes
crates/openshell-prover/src/lib.rs:31-32Testing
mise run pre-commitpassesChecklist