-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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.
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
fix(op-proposer): Missing CheckRequired Invocation #5635
Conversation
|
✅ Deploy Preview for opstack-docs canceled.
|
This generally looks right to me |
This PR has been added to the merge queue, and will be merged soon. |
This PR is next in line to be merged, and will be merged as soon as checks pass. |
1 similar comment
This PR is next in line to be merged, and will be merged as soon as checks pass. |
Description
Following pr #5634, I found the
op-proposer
was missing theCheckRequired
flags check at the beginning of the main l2 output submitter function.I believe it should follow the batch submitter logic.