Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove sequencer from RuntimeTxHook #1831

Merged
merged 1 commit into from
Feb 7, 2025
Merged

Remove sequencer from RuntimeTxHook #1831

merged 1 commit into from
Feb 7, 2025

Conversation

jfldde
Copy link
Contributor

@jfldde jfldde commented Feb 6, 2025

No description provided.

@jfldde jfldde requested a review from eyusufatik as a code owner February 6, 2025 22:59
@auto-assign auto-assign bot requested a review from rakanalh February 6, 2025 22:59
@codecov Codecov
Copy link

codecov bot commented Feb 6, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.6%. Comparing base (d1d6dd4) to head (d469b63).
Report is 4 commits behind head on nightly.

Additional details and impacted files
Files with missing lines Coverage Δ
crates/citrea-stf/src/hooks_impl.rs 38.4% <ø> (ø)
...module-system/sov-modules-stf-blueprint/src/lib.rs 47.5% <ø> (ø)
...tem/sov-modules-stf-blueprint/src/stf_blueprint.rs 73.0% <ø> (-0.6%) ⬇️

... and 13 files with indirect coverage changes

@eyusufatik eyusufatik merged commit f9b0c3f into nightly Feb 7, 2025
14 checks passed
@eyusufatik eyusufatik deleted the remove-seq-from-rt-tx-hook branch February 7, 2025 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eyusufatik eyusufatik eyusufatik approved these changes

@rakanalh rakanalh Awaiting requested review from rakanalh

Assignees

@jfldde jfldde

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jfldde @eyusufatik