Skip to content

[CHIA-4257] optimize parse conditions - #1464

Draft
arvidn wants to merge 2 commits into
mainfrom
parse-reuse-conditions
Draft

[CHIA-4257] optimize parse conditions#1464
arvidn wants to merge 2 commits into
mainfrom
parse-reuse-conditions

Conversation

@arvidn

@arvidn arvidn commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

This adds a fair amount of complexity. I'm not convinced that it's worth it.

@arvidn
arvidn force-pushed the parse-reuse-conditions branch from 4f533c6 to 5c698eb Compare June 10, 2026 14:21
@coveralls-official

Copy link
Copy Markdown

Coverage Report for CI Build 27282967773

Coverage increased (+0.8%) to 82.399%

Details

  • Coverage increased (+0.8%) from the base build.
  • Patch coverage: 52 uncovered changes across 2 files (1114 of 1166 lines covered, 95.54%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
crates/chia-consensus/src/conditions.rs 1158 1107 95.6%
crates/chia-consensus/src/run_block_generator.rs 4 3 75.0%
Total (5 files) 1166 1114 95.54%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 20044
Covered Lines: 16516
Line Coverage: 82.4%
Coverage Strength: 11272037.25 hits per line

💛 - Coveralls

@danieljperry danieljperry changed the title optimize parse conditions [CHIA-4257] optimize parse conditions Jun 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant