Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

fix(derive): Small Fixes and Span Batch Validation Fix#139

Merged
refcell merged 3 commits intomainfrom
refcell/remove-dead-code-restrictions
Apr 24, 2024
Merged

fix(derive): Small Fixes and Span Batch Validation Fix#139
refcell merged 3 commits intomainfrom
refcell/remove-dead-code-restrictions

Conversation

@refcell
Copy link
Copy Markdown
Contributor

@refcell refcell commented Apr 24, 2024

Description

Removes some erroneous #[allow(dead_code)] declarations across the kona-derive crate and fixes span batch validation to set the parent block number. It previously fallibly shadowed the parent_num variable in a lower block.

Copy link
Copy Markdown
Contributor Author

refcell commented Apr 24, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @refcell and the rest of your teammates on Graphite Graphite

@refcell refcell changed the title fix(derive): small fixes across the codebase fix(derive): Small Fixes and Span Batch Validation Fix Apr 24, 2024
@refcell refcell requested a review from clabby April 24, 2024 21:49
@refcell refcell added A-proof Area: proof crates K-fix Kind: fix labels Apr 24, 2024 — with Graphite App
@refcell refcell marked this pull request as ready for review April 24, 2024 21:50
pub fn to_raw_span_batch(
&self,
origin_changed_bit: u8,
_origin_changed_bit: u8,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

hmm.. will look into this

Co-authored-by: clabby <ben@clab.by>
@refcell refcell added this pull request to the merge queue Apr 24, 2024
Merged via the queue into main with commit c0020f8 Apr 24, 2024
@github-actions github-actions bot mentioned this pull request Apr 24, 2024
This was referenced May 29, 2024
This was referenced Jun 6, 2024
This was referenced Jun 16, 2024
@clabby clabby deleted the refcell/remove-dead-code-restrictions branch July 2, 2024 03:39
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
* fix(derive): small fixes across the codebase

* fix(batch): span batch validation

* Update batch.rs

Co-authored-by: clabby <ben@clab.by>

---------

Co-authored-by: clabby <ben@clab.by>
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
* fix(derive): small fixes across the codebase

* fix(batch): span batch validation

* Update batch.rs

Co-authored-by: clabby <ben@clab.by>

---------

Co-authored-by: clabby <ben@clab.by>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A-proof Area: proof crates K-fix Kind: fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants