devtools: test_utxo_snapshots.sh sleep cleanup and documentation #28631

pull fjahr wants to merge 1 commits into bitcoin:master from fjahr:2023-10-au-shell-sleep changing 1 files +6 −6
  1. fjahr commented at 11:01 am on October 10, 2023: contributor
    There were bare sleep statements in the script where it was unclear why they were needed and I think they could also be replaced by existing helpers.
  2. devtools: test_utxo_snapshots.sh sleep cleanup and documentation 36a3004a41
  3. DrahtBot commented at 11:02 am on October 10, 2023: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK Sjors, pablomartin4btc

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #28628 (lint: Include test_utxo_snapshots in lint_shell, fix linter errors by fjahr)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  4. Sjors commented at 7:42 am on October 11, 2023: member
    utACK 36a3004a41aea58f50f3348c5de4eb5a23268788
  5. fanquake requested review from jamesob on Oct 12, 2023
  6. fanquake merged this on Oct 13, 2023
  7. fanquake closed this on Oct 13, 2023


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: 2024-07-03 07:12 UTC

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