Skip to content

Conversation

@infrmtcs
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Dec 26, 2025

Codecov Report

❌ Patch coverage is 76.74419% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.20%. Comparing base (cc180b8) to head (e88c8e6).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
core/block_transaction.go 76.74% 5 Missing and 5 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3348      +/-   ##
==========================================
+ Coverage   76.18%   76.20%   +0.02%     
==========================================
  Files         349      350       +1     
  Lines       33134    33177      +43     
==========================================
+ Hits        25244    25284      +40     
- Misses       6088     6089       +1     
- Partials     1802     1804       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@infrmtcs infrmtcs added the disable-deploy-test We don't want to run deploy tests with this PR because it might affect our development environment. label Dec 26, 2025
@infrmtcs infrmtcs force-pushed the dat/block-transactions branch from f208739 to d8b4ae2 Compare December 28, 2025 14:40
@infrmtcs infrmtcs force-pushed the dat/block-transactions branch from d8b4ae2 to d3f3533 Compare December 28, 2025 16:58
@infrmtcs infrmtcs force-pushed the dat/closer branch 2 times, most recently from d3b8bf5 to 12b90d1 Compare December 29, 2025 15:20
Base automatically changed from dat/closer to main December 29, 2025 16:00
@infrmtcs infrmtcs force-pushed the dat/block-transactions branch 2 times, most recently from 8367591 to 492d696 Compare December 30, 2025 08:48
@infrmtcs infrmtcs changed the base branch from main to dat/adapters December 30, 2025 08:50
Base automatically changed from dat/adapters to main December 30, 2025 11:27
@infrmtcs infrmtcs force-pushed the dat/block-transactions branch from 492d696 to 172042a Compare December 30, 2025 15:26
Copy link
Collaborator

@rodrodros rodrodros left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@EgeCaner EgeCaner left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@infrmtcs infrmtcs removed the disable-deploy-test We don't want to run deploy tests with this PR because it might affect our development environment. label Jan 6, 2026
@infrmtcs infrmtcs force-pushed the dat/block-transactions branch from 172042a to e88c8e6 Compare January 6, 2026 14:53
@infrmtcs infrmtcs merged commit a4a60ba into main Jan 6, 2026
19 of 25 checks passed
@infrmtcs infrmtcs deleted the dat/block-transactions branch January 6, 2026 15:38
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.

4 participants