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

Move l2-block-publisher.test.ts to e2e and re-enable it #458

Closed
benesjan opened this issue May 4, 2023 · 0 comments · Fixed by #463
Closed

Move l2-block-publisher.test.ts to e2e and re-enable it #458

benesjan opened this issue May 4, 2023 · 0 comments · Fixed by #463
Assignees
@benesjan

Comments

@benesjan
Copy link
Contributor

benesjan commented May 4, 2023 •
edited
Loading

This will require a larger refactor since the test currenlty uses ethereum.js and it will need to use Viem.

Wait for #431 to be merged to avoid unnecessary conflicts.
@github-project-automation github-project-automation bot moved this to Todo in A3 May 4, 2023
@benesjan benesjan self-assigned this May 4, 2023
@LHerskind LHerskind mentioned this issue May 4, 2023
Merged
6 tasks
@benesjan benesjan moved this from Todo to In Progress in A3 May 4, 2023
@benesjan benesjan mentioned this issue May 4, 2023
Merged
6 tasks
@github-project-automation github-project-automation bot moved this from In Progress to Done in A3 May 5, 2023
@iAmMichaelConnor iAmMichaelConnor removed this from A3 Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benesjan benesjan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Moving publisher test to e2e and re-enabling it AztecProtocol/aztec-packages
1 participant
@benesjan