-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
refactor: Rename miniblock to l2 block – base + state keeper #1696
Conversation
da7372f
to
41527ba
Compare
May be obvious: if this approach is OK, I plan to create smaller follow-up PRs to change naming in other components. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
I love the idea. I wonder, how can we backfill things that are leftover (as you mentioned DB, APIs, etc.) This is a question for a broader group, though. cc: @RomanBrodetski |
## What ❔ Renames "miniblock" to "L2 block" in libraries (i.e., `core/lib` crates with the exception of `zksync_core`). ## Why ❔ See #1696 for reasoning. ## Checklist - [x] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [x] Tests for the changes have been added / updated. - [x] Documentation comments have been added / updated. - [x] Code has been formatted via `zk fmt` and `zk lint`. - [x] Spellcheck has been run via `zk spellcheck`. - [x] Linkcheck has been run via `zk linkcheck`.
## What ❔ Renames "miniblock" to "L2 block" in core components (ones remaining in `zksync_core`). ## Why ❔ See #1696 for reasoning. ## Checklist - [x] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [x] Tests for the changes have been added / updated. - [x] Documentation comments have been added / updated. - [x] Code has been formatted via `zk fmt` and `zk lint`. - [x] Spellcheck has been run via `zk spellcheck`. - [x] Linkcheck has been run via `zk linkcheck`.
What ❔
Renames "miniblock" to "L2 block" in types, DAL and state keeper.
Why ❔
Checklist
zk fmt
andzk lint
.zk spellcheck
.zk linkcheck
.