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.

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

[DO NOT MERGE] Zkos dev #530

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

[DO NOT MERGE] Zkos dev #530

wants to merge 7 commits into from

Conversation

mm-zk
Copy link
Collaborator

@mm-zk mm-zk commented Jan 13, 2025

What 💻

  • First thing updated with this PR
  • Second thing updated with this PR
  • Third thing updated with this PR

Why ✋

  • Reason why first thing was added to PR
  • Reason why second thing was added to PR
  • Reason why third thing was added to PR

Evidence 📷

Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

mm-zk and others added 6 commits January 15, 2025 14:45
* ugly hacks to return the witness

* compute witness correctly

* passing zkconfig along

* moved zkos into zkos_config

* moved zkos namespace to proper place

* added docs

* put dependency back on main branch
* refactor: reduce usage of `ForkStorage` outside `inner` module (#539)

* reduce usage of `ForkStorage` outside `inner` module

* adapt for zkos

* fix destination path (#541)

* fix: anvil-zksync able to start even if the port is busy (#542)

* fix: closes #512 - start instance even if port is busy

* chore: apply make lint:fix

* chore: refactor test to make use of anvil-zksync instance

* chore: remove no longer needed deps

* chore: remove no longer needed deps

* chore: bump actions/upload-artifact: v3 to v4 due to CI fail

* chore: bump actions/upload-artifact / download: v3 to v4 due to CI fail

* compilation fixes

---------

Co-authored-by: Daniyar Itegulov <[email protected]>
Co-authored-by: Nisheeth Barthwal <[email protected]>
Co-authored-by: Dustin Brickwood <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant