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.

Sign up for GitHub

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

Jump to bottom

Release v25 protocol defense #774

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

Release v25 protocol defense #774

wants to merge 48 commits into from

Conversation

koloz193
Copy link
Contributor

@koloz193 koloz193 commented Sep 5, 2024

What ❔

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.

Raid5594 and others added 30 commits April 24, 2024 15:36
@Raid5594
extend utils & update compilation process (#396)
4c0e566 
Co-authored-by: Raid Ateir <[email protected]>
@neotheprogramist @Uacias @matzayonc
NonceHolder Tests (#176)
3601648 
Co-authored-by: Uacias <[email protected]>
Co-authored-by: Mateusz Zając <[email protected]>
Co-authored-by: Mateusz Zając <[email protected]>
@koloz193
chore(scripts): remove unconditionally removed facets (#448)
2840540
@koloz193
chore(contracts): remove usage of ergs (#454)
622638a
@koloz193
chore(verifier): update generation to include vk hash (#453)
bd9bec5
@koloz193
chore(contract): applied new solhint rules across l1, l2, and system …
994897b 
…contracts (#473)
@koloz193
Merge branch 'release-v23' into dev
8e4c291
@koloz193
tests and fix merge conflict
d97d8cd
@koloz193
fix merge conflicts and bump hashes
704b41b
@koloz193
fixed lint
ee5667e
@koloz193
update system contracts hashes and verifier hash
5c4525a
@koloz193
update forge-std
131f823
@StanislavBreadless
Merge pull request #508 from matter-labs/zk-merge-release-23-dev
360a918 
chore(contracts): merge release 23 into dev
@StanislavBreadless
sync with dev
680b252
@StanislavBreadless
Merge pull request #523 from matter-labs/sb-merge-main-to-dev
8df3025 
Merge main to dev
@benceharomi @StanislavBreadless
chore: zksync-ethers 5.8.0-beta.5 (#387)
311b82e 
Co-authored-by: Stanislav Breadless <[email protected]>
@dimazhornyk
feat: check diamond cut hash locally (#525)
ab2f13d
@StanislavBreadless
Fix incorrect clause (#535)
7d7af53
@vladbochok @ly0va
@kelemeno @StanislavBreadless @juan518munoz @benceharomi @Deniallugo
Adding doc comments to the chain contracts (#530)
100b3bf 
Signed-off-by: Danil <[email protected]>
Co-authored-by: Lyova Potyomkin <[email protected]>
Co-authored-by: kelemeno <[email protected]>
Co-authored-by: Stanislav Bezkorovainyi <[email protected]>
Co-authored-by: Raid Ateir <[email protected]>
Co-authored-by: kelemeno <[email protected]>
Co-authored-by: Jmunoz <[email protected]>
Co-authored-by: Bence Haromi <[email protected]>
Co-authored-by: Danil <[email protected]>
@vladbochok
Add CI coverage (reopenned to dev) (#543)
dc7bd13
@neotheprogramist @tommysr
Mailbox unit testing (#489)
8fec7ec 
Co-authored-by: tommysr <[email protected]>
@saxenism
Revert "Mailbox unit testing (#489)" (#558)
116206c 
CI checks were failing in the earlier branch
@artmakh
chore(ci): Additional permissions for CI token (#565)
fb3a5d8
@tommysr
Mailbox unit testing (#559)
1c1b93b 
Failing CI check is for coverage and for external contributors, they are not permissioned enough to run this check. The reason being Github doesn't allow granting anything except read for tokens, when PR is from fork https://docs.github.com/en/actions/security-guides/automatic-token-authentication#permissions-for-the-github_token
@benceharomi
chore: zksync-ethers to stable v5.9.0 (#567)
c9a8a68
@kelemeno
remove outdated folder (#601)
bc98e4b
@koloz193
Merge branch 'main' into zk-sync-main-dev
c808b1f
@StanislavBreadless
Merge pull request #678 from matter-labs/zk-sync-main-dev
5b8d6ae 
Merge Main into Dev
@StanislavBreadless
sync main with dev
3161150
@StanislavBreadless
Merge pull request #681 from matter-labs/sb-merge-main-into-dev
0b80914 
Merge main into dev
Deniallugo and others added 18 commits August 13, 2024 20:33
@Deniallugo
Merge branch 'main' into deniallugo-merge-main
1812c31
@StanislavBreadless
Merge pull request #696 from matter-labs/deniallugo-merge-main
d0d735c 
merge main
@neotheprogramist @tommysr
Bridgehub Test Coverage Increase (#494)
118f081 
Co-authored-by: tommysr <[email protected]>
@Deniallugo @vladbochok @perekopskiy
feat(foundry): use foundry zksync (dev) (#551)
7f4f460 
Signed-off-by: Danil <[email protected]>
Co-authored-by: vladbochok <[email protected]>
Co-authored-by: Vlad Bochok <[email protected]>
Co-authored-by: perekopskiy <[email protected]>
Co-authored-by: perekopskiy <[email protected]>
@moshababo @aakoshh
@brunoffranca @vladbochok @RomanBrodetski
feat(consensus): add L2 registry contract (BFT-434) (#555)
80e8234 
Co-authored-by: Akosh Farkash <[email protected]>
Co-authored-by: Bruno França <[email protected]>
Co-authored-by: Vlad Bochok <[email protected]>
Co-authored-by: Roman Brodetski <[email protected]>
Co-authored-by: vladbochok <[email protected]>
@pompon0
Revert: "feat(consensus): add L2 registry contract (BFT-434)" (#717)
2c8a5fa
@koloz193 @benceharomi
Merge Protocol Defense Tasks into Dev (#518)
8be87b2 
Co-authored-by: Bence Haromi <[email protected]>
@benceharomi
chore: foundry config cleanup (#722)
7b488e6
@IAvecilla
Update solidity contracts version (#688)
2395aed
@StanislavBreadless @koloz193
chore(config): Fix server build with symlinks (#740)
a36c6c2 
Co-authored-by: Zach Kolodny <[email protected]>
@dimazhornyk
feat: add a script to prepare chain registration calldata (#671)
8d9240f
@benceharomi
feat(l2): solidity 0.8.24 and zksolc 1.5.0 (#743)
1e0df10
@benceharomi
feat(l1): added missing foundry lib symlinks (#741)
f2d553b
@benceharomi
docs: update zkSync casing to ZKsync (#733)
383e1b0
@koloz193
chore(contracts): add missing spdx licenses (#732)
dba0212
@koloz193
fix(scrips): read bytecode from foundry not hardhat (#750)
874bc6b
@Raid5594
Small PR to Fix "Ethers" and "ETH" encoding (#762)
7f2c27a 
Co-authored-by: Raid Ateir <[email protected]>
@perekopskiy
Set foundry optimizer_runs (#770)
3d6e02f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Deniallugo Deniallugo Awaiting requested review from Deniallugo Deniallugo will be requested when the pull request is marked ready for review Deniallugo is a code owner

@vladbochok vladbochok Awaiting requested review from vladbochok vladbochok will be requested when the pull request is marked ready for review vladbochok is a code owner

@StanislavBreadless StanislavBreadless Awaiting requested review from StanislavBreadless StanislavBreadless will be requested when the pull request is marked ready for review StanislavBreadless is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.