test: remove unnecessary m_best_header setting hack in feature_assumeutxo.py #32746

pull theStack wants to merge 1 commits into bitcoin:master from theStack:202506-test-assumeutxo_remove_hack_for_mbestheader changing 1 files +0 −6
  1. theStack commented at 12:50 PM on June 13, 2025: contributor

    The mentioned bug (issue #26245) has been fixed in PR #30666, so the workaround is not needed anymore.

  2. test: remove unnecessary m_best_header setting hack in feature_assumeutxo.py
    The mentioned bug has been fixed in PR #30666,
    so the workaround is not needed anymore.
    206bc05e62
  3. DrahtBot commented at 12:50 PM on June 13, 2025: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32746.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK fjahr, janb84, mzumsande

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

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  4. DrahtBot added the label Tests on Jun 13, 2025
  5. theStack renamed this:
    test: remove unnecessary m_best_header setting hack in feature_assume_utxo.py
    test: remove unnecessary m_best_header setting hack in feature_assumeutxo.py
    on Jun 13, 2025
  6. fjahr commented at 2:16 PM on June 13, 2025: contributor

    ACK 206bc05e62c6a71c328eaab90fd865f46328642f

  7. janb84 commented at 2:26 PM on June 13, 2025: contributor

    ACK 206bc05e62c6a71c328eaab90fd865f46328642f

    The issue is resolved, the "hack" is not needed anymore.

    • Validated that the test still runs / passed ✅
  8. mzumsande commented at 2:37 PM on June 13, 2025: contributor

    utACK 206bc05e62c6a71c328eaab90fd865f46328642f

  9. fanquake merged this on Jun 13, 2025
  10. fanquake closed this on Jun 13, 2025

  11. theStack deleted the branch on Jun 13, 2025
  12. bitcoin deleted a comment on Jun 13, 2025
  13. stringintech referenced this in commit f19e7819e9 on Jul 19, 2025
  14. alexanderwiederin referenced this in commit 9997c3d993 on Jul 25, 2025
  15. alexanderwiederin referenced this in commit 8d6ebb9c4e on Jul 28, 2025
  16. alexanderwiederin referenced this in commit 533a4585b5 on Jul 28, 2025
  17. yuvicc referenced this in commit 22f55cf11d on Aug 26, 2025
  18. bug-castercv502 referenced this in commit d4bd8a1259 on Sep 28, 2025
  19. stickies-v referenced this in commit a19c56cd7c on Nov 4, 2025
  20. Kino1994 referenced this in commit b1a5eb957a on Jun 28, 2026
  21. BigcoinBGC referenced this in commit 6e3be238de on Jun 30, 2026
  22. PastaPastaPasta referenced this in commit 78d88083d8 on Jul 12, 2026
  23. PastaPastaPasta referenced this in commit f359fdbce6 on Jul 12, 2026
  24. PastaPastaPasta referenced this in commit 7459dadf6f on Jul 12, 2026
  25. PastaPastaPasta referenced this in commit f8520dae66 on Jul 12, 2026
  26. PastaPastaPasta referenced this in commit d945b2b411 on Jul 14, 2026
  27. PastaPastaPasta referenced this in commit 4a0fcf61f1 on Jul 14, 2026
  28. PastaPastaPasta referenced this in commit 69236c78fc on Jul 14, 2026
  29. PastaPastaPasta referenced this in commit db6bac79f6 on Jul 14, 2026
  30. PastaPastaPasta referenced this in commit 930546d6d1 on Jul 20, 2026
  31. PastaPastaPasta referenced this in commit 70728886be on Jul 20, 2026
  32. bitcoin locked this on Jul 30, 2026

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-08-01 10:51 UTC

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