Skip to content

Conversation

arvidn
Copy link
Contributor

@arvidn arvidn commented Aug 1, 2025

Purpose:

This PR is best reviewed one commit at a time. It updates chia_rs to the latest version:

  1. correct the return value from get_spends_for_trusted_block() which matt fixed recently.
  2. remove casts that are no longer necessary, with the fixed type stubs
  3. adjust a test to expect a slightly different error message, related to the work on error codes in clvm_rs Jack did.

@arvidn arvidn force-pushed the bump-chia-rs-0.28 branch from 30ea6b0 to 0beaa56 Compare August 1, 2025 17:54
@altendky altendky mentioned this pull request Aug 1, 2025
21 tasks
@arvidn arvidn force-pushed the bump-chia-rs-0.28 branch from 0beaa56 to 0b7ce5d Compare August 1, 2025 20:07
@arvidn arvidn added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Aug 1, 2025
@arvidn arvidn marked this pull request as ready for review August 1, 2025 20:11
@arvidn arvidn requested review from a team and altendky as code owners August 1, 2025 20:11
Copy link
Contributor

github-actions bot commented Aug 2, 2025

File Coverage Missing Lines
chia/cmds/show_funcs.py 60.0% lines 51, 62
chia/full_node/full_node_rpc_api.py 50.0% lines 486
Total Missing Coverage
20 lines 3 lines 85%

@arvidn arvidn requested a review from wjblanke August 4, 2025 18:38
Copy link
Contributor

@wjblanke wjblanke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aok

@Starttoaster Starttoaster merged commit 04eef67 into main Aug 5, 2025
518 of 521 checks passed
@Starttoaster Starttoaster deleted the bump-chia-rs-0.28 branch August 5, 2025 18:20
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.

4 participants