ligurio / elle-cli

The command-line frontend to transactional consistency checkers for black-box databases
Other
27 stars 5 forks source link

Fix exit code when history is not valid #54

Closed ligurio closed 2 years ago

ligurio commented 2 years ago
mprimi commented 2 years ago

Thank you for the quick turnaround on this. Personally I like this, but notice this behavior is not what the README describes:

prints out the names of all files you asked it to check, followed by a tab, and then whether the history was valid.

Could update the README to say "terminates at the first non-valid", or could still run all of them and return non-zero if any of them is non-valid.

ligurio commented 2 years ago

Could update the README to say "terminates at the first non-valid", or could still run all of them and return non-zero if any of them is non-valid.

Patch has been updated, now it checks the whole list of histories and then exits.