Skip to content

Comments

feat: Implement proposer component#2947

Merged
infrmtcs merged 1 commit intomainfrom
dat/proposer
Jul 7, 2025
Merged

feat: Implement proposer component#2947
infrmtcs merged 1 commit intomainfrom
dat/proposer

Conversation

@infrmtcs
Copy link
Contributor

@infrmtcs infrmtcs commented Jul 1, 2025

No description provided.

@codecov
Copy link

codecov bot commented Jul 1, 2025

Codecov Report

Attention: Patch coverage is 77.50000% with 36 lines in your changes missing coverage. Please review.

Project coverage is 72.25%. Comparing base (f950632) to head (67038c1).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
consensus/proposer/proposer.go 73.68% 23 Missing and 7 partials ⚠️
builder/executor.go 0.00% 2 Missing and 1 partial ⚠️
builder/state.go 92.50% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2947      +/-   ##
==========================================
+ Coverage   72.19%   72.25%   +0.06%     
==========================================
  Files         262      263       +1     
  Lines       28368    28572     +204     
==========================================
+ Hits        20479    20645     +166     
- Misses       6496     6526      +30     
- Partials     1393     1401       +8     

☔ 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 force-pushed the dat/proposer branch 2 times, most recently from b628000 to 4d8ed3d Compare July 2, 2025 06:11
@rianhughes rianhughes self-requested a review July 7, 2025 07:58
@rianhughes
Copy link
Contributor

rianhughes commented Jul 7, 2025

I think we should really keep track of the fact our pending block does not have proper read-write controls. I've created an issue to track this here #2957. This is a bigger issue that is outside the scope of this PR imo

@infrmtcs infrmtcs merged commit d96d62b into main Jul 7, 2025
20 checks passed
@infrmtcs infrmtcs deleted the dat/proposer branch July 7, 2025 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants