bitcoindevkit / bdk-cli / 15421780355

11%

push

github

Merge bitcoindevkit/bdk-cli#173: Fix exit code

304729b17 fix: use exit code 1 in case of error (Vadim Anufriev)

Pull request description:

  Fix for https://github.com/bitcoindevkit/bdk-cli/issues/170.

  I think this is the simplest way to do it.
  It's also possible to change the main signature to return `Result`, which should provide similar behavior, but this option isn't compatible with the `#[maybe_async]` macro, so I decided not to complicate things.

  ### Checklists

  #### All Submissions:

  * [x] I've signed all my commits
  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk-cli/blob/master/CONTRIBUTING.md)
  * [x] I ran `cargo fmt` and `cargo clippy` before committing

  #### Bugfixes:

  * [ ] This pull request breaks the existing API
  * [ ] I've added tests to reproduce the issue which are now passing
  * [x] I'm linking the issue being fixed by this PR

Top commit has no ACKs.

Tree-SHA512: 52ce4292cdc90647a25a36ea273f585cf7d

0 of 2 new or added lines in 1 file covered. (0.0%)

25 of 926 relevant lines covered (2.7%)

0.05 hits per line