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

feat: Store l2 pubdata in memory #43

Closed
wants to merge 1 commit into from
Closed

feat: Store l2 pubdata in memory #43

wants to merge 1 commit into from

Conversation

toni-calvin
Copy link

What ❔

Why ❔

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zk fmt and zk lint.
  • Spellcheck has been run via zk spellcheck.

@ilitteri
Copy link
Collaborator

Not needed for the moment. We're not going to work with another data layer model.

@ilitteri ilitteri closed this Jan 16, 2024
@ilitteri ilitteri deleted the store-l2-pubdata-memory branch February 9, 2024 15:58
Oppen pushed a commit that referenced this pull request Feb 9, 2024
@benceharomi
feat: use boojum-integration branch of in-memory node for testing CI (#…
0d38b70 
…43)

* ci: using boojum branch of test node

* test: reenable temporarily disabled tests

* ci: test node in background

* ci: caching for era-test-node

* chore: downgrading hardhat version to fix test execution

* ci: ci to run on dev and main push

* chore: set hardhat to fix v2.16.0

* ci: print era_test_node logs

* ci: change tag to commit SHA of dependency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
external-contribution
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@toni-calvin @ilitteri