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

Feature/#1615 rlp #1617

Merged
merged 8 commits into from
Feb 8, 2023
Merged

Feature/#1615 rlp #1617

merged 8 commits into from
Feb 8, 2023

Conversation

ARR552
Copy link
Contributor

@ARR552 ARR552 commented Feb 1, 2023

Closes #1615

What does this PR do?

This Pr adds the rlp tx that failed on public as a test

Reviewers

Main reviewers:

@cla-bot cla-bot bot added the cla-signed label Feb 1, 2023
@ARR552 ARR552 merged commit 9d2b312 into develop Feb 8, 2023
@ARR552 ARR552 deleted the feature/#1615_rlp branch February 8, 2023 10:26
tclemos pushed a commit that referenced this pull request Feb 14, 2023
@ARR552 @tclemos
Feature/#1615 rlp (#1617)
872315f 
* script

* rlp executor encoding test

* linter

* fix public config

* update prover version

* remove TrustedSequencerURI

* fix script log
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arnaubennassar arnaubennassar arnaubennassar approved these changes

@ToniRamirezM ToniRamirezM ToniRamirezM approved these changes

@tclemos tclemos Awaiting requested review from tclemos tclemos is a code owner

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

investigate rlp and add tests
3 participants
@ARR552 @arnaubennassar @ToniRamirezM