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

docs(zkstack_cli): update docs to mention foundry zksync #3134

Merged
merged 71 commits into from
Nov 13, 2024

Conversation

manuelmauro
Copy link
Contributor

@manuelmauro manuelmauro commented Oct 21, 2024

What ❔

Update docs to mention foundry zksync, and other changes to the CLI.

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 zkstack dev fmt and zkstack dev lint.

@manuelmauro manuelmauro force-pushed the manuel-update-docs-to-mention-foundry-zksync branch from 8dc044e to bb58441 Compare October 21, 2024 10:18
@manuelmauro manuelmauro marked this pull request as ready for review October 22, 2024 09:11
@Deniallugo Deniallugo requested review from popzxc and mm-zk October 22, 2024 15:40
docs/guides/advanced/01_initialization.md Outdated Show resolved Hide resolved
docs/guides/development.md Outdated Show resolved Hide resolved
docs/guides/development.md Outdated Show resolved Hide resolved
docs/guides/launch.md Outdated Show resolved Hide resolved
docs/guides/launch.md Show resolved Hide resolved
docs/guides/launch.md Outdated Show resolved Hide resolved
@EmilLuta
Copy link
Contributor

Could you please cover https://github.com/matter-labs/zksync-era/tree/main/prover/docs as well? Especially https://github.com/matter-labs/zksync-era/blob/main/prover/docs/03_launch.md.

docs/guides/launch.md Outdated Show resolved Hide resolved
docs/guides/launch.md Outdated Show resolved Hide resolved
Copy link
Contributor

@sarahschwartz sarahschwartz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great updates! Just have some grammar nits

docs/guides/advanced/01_initialization.md Outdated Show resolved Hide resolved
docs/guides/advanced/01_initialization.md Outdated Show resolved Hide resolved
docs/guides/advanced/01_initialization.md Outdated Show resolved Hide resolved
docs/guides/development.md Outdated Show resolved Hide resolved
docs/guides/development.md Outdated Show resolved Hide resolved
docs/guides/launch.md Outdated Show resolved Hide resolved
manuelmauro and others added 6 commits November 6, 2024 12:14
Co-authored-by: Sarah Schwartz <58856580+sarahschwartz@users.noreply.github.com>
Co-authored-by: Sarah Schwartz <58856580+sarahschwartz@users.noreply.github.com>
Co-authored-by: Sarah Schwartz <58856580+sarahschwartz@users.noreply.github.com>
Co-authored-by: Sarah Schwartz <58856580+sarahschwartz@users.noreply.github.com>
Co-authored-by: Sarah Schwartz <58856580+sarahschwartz@users.noreply.github.com>
Co-authored-by: Sarah Schwartz <58856580+sarahschwartz@users.noreply.github.com>
@manuelmauro
Copy link
Contributor Author

Great updates! Just have some grammar nits

Thank you! @sarahschwartz

docs/guides/launch.md Outdated Show resolved Hide resolved
docs/guides/setup-dev.md Show resolved Hide resolved
@manuelmauro
Copy link
Contributor Author

@Deniallugo I went through the whole dev-setup process within the VM you provided and everything worked just fine!

@Deniallugo Deniallugo enabled auto-merge November 13, 2024 01:40
@Deniallugo Deniallugo added this pull request to the merge queue Nov 13, 2024
Merged via the queue into main with commit 4afbc31 Nov 13, 2024
20 checks passed
@Deniallugo Deniallugo deleted the manuel-update-docs-to-mention-foundry-zksync branch November 13, 2024 01:53
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.

9 participants