Skip to content

feat(protocol): Batch#172

Closed
refcell wants to merge 2 commits intomainfrom
rf/feat/batch-type
Closed

feat(protocol): Batch#172
refcell wants to merge 2 commits intomainfrom
rf/feat/batch-type

Conversation

@refcell
Copy link
Copy Markdown
Collaborator

@refcell refcell commented Oct 15, 2024

Description

kona-derive is nearly decoupled from protocol-level types. The final remaining types are the batch types: single and span batches.

This PR ports over all batch types from kona-derive's batch module, introducing a new minimal trait abstraction for batch validity checks called the BatchValidator. Effectively, it is a subset of the L2ChainProvider trait in kona-derive.

Additional Context

Works on op-rs/kona#695

@refcell refcell changed the title feat: batch type feat(protocol): batch type Oct 15, 2024
@refcell refcell changed the title feat(protocol): batch type feat(protocol): Batch Type Oct 15, 2024
@refcell refcell self-assigned this Oct 24, 2024
### Description

> [!WARNING]
>
> Stacked ontop of #172 

Adds the single batch to op-alloy-protocol.
@refcell refcell changed the title feat(protocol): Batch Type feat(protocol): Batch Oct 25, 2024
Comment thread crates/protocol/src/traits.rs
@refcell refcell added D-do-not-merge Desc: Do not merge D-do-not-review Desc: Not ready for a review labels Oct 25, 2024
@refcell
Copy link
Copy Markdown
Collaborator Author

refcell commented Oct 27, 2024

This PR is replaced by the stack started in #186

@refcell refcell closed this Oct 29, 2024
@refcell refcell deleted the rf/feat/batch-type branch November 1, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

D-do-not-merge Desc: Do not merge D-do-not-review Desc: Not ready for a review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants