Skip to content

Conversation

@0xValera
Copy link
Contributor

@0xValera 0xValera commented Nov 27, 2025

What ❔

There was still some documentation present in era-contracts repository. Since we try to store all of the docs in zksync-era currently it leads to duplication of docs and negative work whenever someone pushes new docs/changes to era-contracts instead of zksync-era.

This PR moves/corrects all of the docs from era-contracts that was missed here.

This is not a permanent solution, since zksync-era repo will be deprecated eventually, but it's the best solution we agreed upon as protocol team for the time being.

The respective PR in era-contracts is here.

Why ❔

Is this a breaking change?

  • Yes
  • No

Operational changes

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.

Copy link
Member

@zkzoomer zkzoomer left a comment

Choose a reason for hiding this comment

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

Looks good overall, just some suggestions. The dead links CI from era-contracts should also be added in as a new workflow.

@0xValera 0xValera force-pushed the vg/docs-migration-from-era-contracts branch from 7c16d4e to 4285de0 Compare January 5, 2026 12:37
@0xValera 0xValera requested a review from a team as a code owner January 6, 2026 09:58
@0xValera 0xValera force-pushed the vg/docs-migration-from-era-contracts branch from bdc5f42 to 38aa4e7 Compare January 8, 2026 11:35
@0xValera 0xValera added this pull request to the merge queue Jan 12, 2026
Merged via the queue into main with commit 198dd4d Jan 12, 2026
21 checks passed
@0xValera 0xValera deleted the vg/docs-migration-from-era-contracts branch January 12, 2026 12:15
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.

3 participants