Skip to content

Add partial Amsterdam logic#43

Merged
han0110 merged 1 commit intomainfrom
jsign-amsterdam-partial
Apr 28, 2026
Merged

Add partial Amsterdam logic#43
han0110 merged 1 commit intomainfrom
jsign-amsterdam-partial

Conversation

@jsign
Copy link
Copy Markdown
Collaborator

@jsign jsign commented Apr 27, 2026

This PR tries to push into main as much logic as possible from #36, so we keep the diff with that PR as small as possible.

Co-authored-by: Copilot <copilot@github.com>
Comment on lines +283 to +284
// TODO(Amsterdam): use actual BAL that should be received from parameter.
let block_access_list = Tx::try_from(alloy_rlp::encode(BlockAccessList::default()))
Copy link
Copy Markdown
Collaborator Author

@jsign jsign Apr 27, 2026

Choose a reason for hiding this comment

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

The impl of this TODO would be the code happening in the BAL-devnet-3 branch/PR, since current EL dependencies doesn't have the logic to do this here now.

chain_spec.is_prague_active_at_timestamp(sealed_block.timestamp),
)?;

// TODO(Amsterdam): Add Amsterdam-specific validation.
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Ditto, since current EL branches don't have the code to check this.

@jsign jsign requested a review from han0110 April 27, 2026 15:06
@jsign jsign marked this pull request as ready for review April 27, 2026 15:06
Copy link
Copy Markdown
Collaborator

@han0110 han0110 left a comment

Choose a reason for hiding this comment

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

LGTM

@han0110 han0110 merged commit 02da61a into main Apr 28, 2026
41 checks passed
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.

2 participants