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

4844 int opt #96

Closed
wants to merge 87 commits into from
Closed

4844 int opt #96

wants to merge 87 commits into from

Conversation

yahgwai
Copy link
Contributor

@yahgwai yahgwai commented Nov 3, 2023 •
edited
Loading

Adds 4844 blob reading

  • Reads data from blob
  • Submits batch spending report of blobbasefee

Nitro changes

  • New seq inbox function to submit blobs to
  • Batch spending reports use a new gas price value

@yahgwai
Initial 4844 commit with tests
7bacbf0
@yahgwai
Merge branch 'develop' into 4844-int
36b0925
@yahgwai
Merge from develop
6092d89
@yahgwai
Updated formatting
233826f
@yahgwai
Recommented yul compile
557835e
@yahgwai
Update unit tests
a7286ef
@yahgwai
Added ci for 4844 tests
190107b
@yahgwai
test yes
1e22c1c
@yahgwai
Updated to use deneb integration branch
81308cf
@yahgwai
Linting updates
5f4e912
@yahgwai
Updated validator tests
597bd26
@yahgwai
Seq inbox stub update
7073bc9
@yahgwai
Refactored sequencer inbox
8b379e9
@yahgwai
Use args test branch
888d13e
@yahgwai
Updated storage dot
a238e59
@yahgwai
Formatting
48e3bfe
@yahgwai
Test formatting
2acdc24
@yahgwai
Removed formatting
4b061cb
@yahgwai
Added no-token-bridge arg
24f4381
@yahgwai
Updated function name in ts
a89fed5
@yahgwai
lint tests
8038022
@yahgwai
Added test get network
4207de1
@yahgwai
Test change
4109298
@yahgwai
Exec sync curl
db6c1bb
@yahgwai
Updated to main
617a64a
@yahgwai
Test use blocking branch
8add810
@yahgwai
Test exec on 8545
ea83596
@yahgwai
Moved up the getnetwork command
0dffda5
@yahgwai
Added additional wait
1a39b7a
@yahgwai
Check buffer each round
381e7b0
@yahgwai yahgwai marked this pull request as ready for review November 17, 2023 14:04
PlasmaPower
PlasmaPower reviewed Nov 17, 2023
View reviewed changes
src/bridge/SequencerInbox.sol Outdated Show resolved Hide resolved
src/bridge/SequencerInbox.sol Show resolved Hide resolved
@shotaronowhere shotaronowhere mentioned this pull request Nov 28, 2023
Closed
This was referenced Jan 17, 2024
Merged
Draft
@gzeoneth gzeoneth closed this Mar 5, 2024
@gzeoneth gzeoneth deleted the 4844-int-opt branch July 8, 2024 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PlasmaPower PlasmaPower PlasmaPower left review comments

@gvladika gvladika Awaiting requested review from gvladika

@Tristan-Wilson Tristan-Wilson Awaiting requested review from Tristan-Wilson

@gzeoneth gzeoneth Awaiting requested review from gzeoneth

Assignees
No one assigned
Labels
s seq-inbox
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@yahgwai @PlasmaPower @Tristan-Wilson @DZGoldman @gzeoneth