Skip to content

Sequencer injected batch#95

Merged
mandrigin merged 7 commits into
zkevmfrom
sequencer-injected-batch
Feb 6, 2024
Merged

Sequencer injected batch#95
mandrigin merged 7 commits into
zkevmfrom
sequencer-injected-batch

Conversation

@hexoscott

Copy link
Copy Markdown

No description provided.

Comment thread zk/hermez_db/db.go
Comment thread cmd/utils/flags.go Outdated
Comment thread zk/stages/stage_l1_sequencer_sync.go Outdated
Comment thread zk/stages/stage_sequence_execute.go Outdated

@mandrigin mandrigin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

a few minor comments

@hexoscott hexoscott force-pushed the sequencer-injected-batch branch from 2d45a70 to 45cb16f Compare February 1, 2024 15:33
@hexoscott

Copy link
Copy Markdown
Author

comments addressed @mandrigin

@hexoscott

Copy link
Copy Markdown
Author

some fixes to make here after a rebase, will update shortly

@hexoscott

Copy link
Copy Markdown
Author

OK, looking good now and using the correct block info tree root

@hexoscott hexoscott force-pushed the sequencer-injected-batch branch 2 times, most recently from e0472f4 to 4220c86 Compare February 5, 2024 10:05
@hexoscott hexoscott force-pushed the sequencer-injected-batch branch from b926e0d to 772e367 Compare February 5, 2024 14:08
@mandrigin mandrigin merged commit d0dd920 into zkevm Feb 6, 2024
@mandrigin mandrigin deleted the sequencer-injected-batch branch February 6, 2024 08:12
googgoog pushed a commit to googgoog/cdk-erigon that referenced this pull request Apr 2, 2025
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.

4 participants