Refactor bitcoin-cli.cpp #7986

pull tuladhar wants to merge 1 commits into bitcoin:master from tuladhar:patch-1 changing 1 files +1 −1
  1. tuladhar commented at 9:00 AM on May 2, 2016: contributor

    No description provided.

  2. Update bitcoin-cli.cpp e4aa4db802
  3. jonasschnelli added the label Utils and libraries on May 2, 2016
  4. kazcw commented at 3:50 PM on May 2, 2016: contributor

    The commit summary's not very specific - the change does speak for itself when you're looking at a diff but the logs get read a lot more often than the diffs. What about something like "bitcoin-cli.cpp: Use symbolic constant for exit code"

  5. btcdrak commented at 4:08 PM on May 2, 2016: contributor

    Agree with @kazcw, the commit message needs to be explanatory. It's mentioned in the contributing guidelines https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#contributor-workflow

  6. tuladhar commented at 4:49 PM on May 2, 2016: contributor

    Thanks @kazcw

    Closing in favor of #7989

  7. tuladhar closed this on May 2, 2016

  8. tuladhar deleted the branch on May 2, 2016
  9. tuladhar restored the branch on Oct 2, 2016
  10. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-13 18:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me