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

chore: renames era_test_node to anvil-zksync #444

Merged
merged 10 commits into from
Dec 4, 2024

Conversation

@dutterbutter dutterbutter force-pushed the db/rename-era-test-node-to-anvil-zksync branch from 1f200ec to f383ea1 Compare November 28, 2024 15:52
@dutterbutter dutterbutter added the do not merge 🛑 Do not merge this PR until label is removed label Nov 28, 2024
@dutterbutter dutterbutter marked this pull request as ready for review November 28, 2024 20:56
@dutterbutter dutterbutter requested a review from a team as a code owner November 28, 2024 20:56
Copy link
Member

@popzxc popzxc left a comment

Choose a reason for hiding this comment

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

Mostly LGTM. If you will need me to rename the repo today, just ping in tg.

README.md Outdated Show resolved Hide resolved
@dutterbutter dutterbutter force-pushed the db/rename-era-test-node-to-anvil-zksync branch 2 times, most recently from d999ec9 to 4dff617 Compare November 29, 2024 17:26
@dutterbutter dutterbutter added needs review 👓 PR requires a review and removed do not merge 🛑 Do not merge this PR until label is removed labels Dec 2, 2024
Copy link
Contributor

@itegulov itegulov 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, just a few nits I have found.

We should also rename .github/assets/era_test_node_banner_{dark,light}.png and ideally the contents of the banner itself too (although not that easy so maybe we should remove the files altogether for now?).

There are some references to "zkSync Era In-Memory Node" left which should be renamed too (README.md, spec-tests/tests/lib.rs, src/lib.rs from what I have seen, could be more). Same for "Era Test Node" (many references).

e2e-tests/README.md Outdated Show resolved Hide resolved
scripts/install.sh Show resolved Hide resolved
@dutterbutter dutterbutter force-pushed the db/rename-era-test-node-to-anvil-zksync branch from b30f44e to bd48bb0 Compare December 3, 2024 13:15
@dutterbutter dutterbutter requested a review from itegulov December 3, 2024 13:16
@dutterbutter
Copy link
Collaborator Author

@itegulov went through again and updated. Please review at your convenience in preparation for tomorrows release 🙏

@dutterbutter dutterbutter merged commit ab71413 into main Dec 4, 2024
14 checks passed
@dutterbutter dutterbutter deleted the db/rename-era-test-node-to-anvil-zksync branch December 4, 2024 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review 👓 PR requires a review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants