Skip to content

Conversation

AmineKhaldi
Copy link
Contributor

@AmineKhaldi AmineKhaldi commented Aug 6, 2025

Purpose:

Make sure we end up using the coin_puzzle_hash index in get_unspent_lineage_info_for_puzzle_hash if the new ff unspent index doesn't exist.

Current Behavior:

New Behavior:

Testing Notes:

@AmineKhaldi AmineKhaldi self-assigned this Aug 6, 2025
@AmineKhaldi AmineKhaldi added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Aug 6, 2025
Copy link

coveralls-official bot commented Aug 6, 2025

Pull Request Test Coverage Report for Build 16804907688

Details

  • 5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
  • 20 unchanged lines in 8 files lost coverage.
  • Overall coverage increased (+0.005%) to 91.449%

Files with Coverage Reduction New Missed Lines %
chia/full_node/full_node_rpc_api.py 1 79.18%
chia/plotters/plotters.py 1 90.94%
chia/rpc/rpc_server.py 1 89.78%
chia/wallet/wallet_node.py 1 87.89%
chia/data_layer/data_layer.py 2 86.58%
chia/util/files.py 2 91.38%
chia/full_node/full_node.py 6 88.51%
chia/plotters/madmax.py 6 44.58%
Totals Coverage Status
Change from base Build 16679525425: 0.005%
Covered Lines: 102530
Relevant Lines: 111994

💛 - Coveralls

@AmineKhaldi AmineKhaldi marked this pull request as ready for review August 6, 2025 17:46
@AmineKhaldi AmineKhaldi requested a review from a team as a code owner August 6, 2025 17:46
@AmineKhaldi AmineKhaldi force-pushed the explicit_ph_idx_fallback_get_unspent_lineage branch from a45c2de to 8a76d90 Compare August 7, 2025 10:00
@AmineKhaldi AmineKhaldi requested a review from arvidn August 7, 2025 10:01
@AmineKhaldi AmineKhaldi force-pushed the explicit_ph_idx_fallback_get_unspent_lineage branch from 8a76d90 to f3c6127 Compare August 7, 2025 12:47
@AmineKhaldi AmineKhaldi requested a review from arvidn August 7, 2025 12:48
@Starttoaster Starttoaster merged commit 6821c48 into Chia-Network:release/2.5.5 Aug 7, 2025
2874 of 2883 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changed Required label for PR that categorizes merge commit message as "Changed" for changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants