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

Adding doc comments to the chain contracts #530

Merged
merged 150 commits into from
Jun 18, 2024
Merged

Adding doc comments to the chain contracts #530

merged 150 commits into from
Jun 18, 2024

Conversation

vladbochok
Copy link
Member

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.

ly0va and others added 30 commits April 2, 2024 15:55
@ly0va
Use Governance contract as STM owner
587fc6f
@ly0va
Fix deploy script with undefined chainId
56cfe4b
@kelemeno
updating hyperchain migration tests
08fcdc3
@kelemeno
updating hyperchain migration tests
fed442f
@kelemeno
modifying hyperchain upgrade
a975413
@kelemeno
creating script for hyperchain upgrade
1cc2c80
@kelemeno
updated upgrade scripts
5eaf3fa
@kelemeno
zk fmt
89ebe8e
@kelemeno
contracts era chain id fix
5cf9582
@kelemeno
zk fmt and linting
34a9cd5
@kelemeno
small fix in deploy script
5a32908
@kelemeno
genesis protocol version from latest p v
a21571b
@kelemeno
updated upgrade tests
30f8956
@kelemeno
Ownable2StepUpgradeable
4befd8a
@kelemeno
fixed typo
c6be4c7
@kelemeno
fixed "changeFeeParams" function
445bcf7
@kelemeno
setValidium mode based on storedBatchHashes
cff36f9
@vladbochok @kelemeno
Fix PR review
8f4c5b0
@kelemeno
fixed currentPendingAdmin log error
ee5ca89
@kelemeno
new events
021b1c0
@kelemeno
gas cost fixes
38147a6
@kelemeno
zk lint
ee7bd5a
@kelemeno
changed STM -> DP functions
db1130f
@kelemeno
zk fmt
78d4951
@vladbochok @kelemeno
Address review
622a801
@vladbochok @kelemeno
Fix review
df9f325
@kelemeno
added onlyOwner
488d15b
@kelemeno
validation issues, OZ L04
bc02bed
@kelemeno
zk fmt
0b93956
@kelemeno
fixed tests
8a5de19
@vladbochok vladbochok changed the title OZ hyper improvements audit to dev Adding doc comments to the chain contracts Jun 12, 2024
@StanislavBreadless StanislavBreadless merged commit 100b3bf into dev Jun 18, 2024
21 checks passed
@StanislavBreadless StanislavBreadless deleted the oz-hyper-improvements-audit branch June 18, 2024 13:01
Yberjon pushed a commit to neotheprogramist/era-contracts that referenced this pull request Aug 12, 2024
@vladbochok @ly0va
@kelemeno @StanislavBreadless @juan518munoz @benceharomi @Deniallugo @Yberjon
Adding doc comments to the chain contracts (matter-labs#530)
bdcf146 
Signed-off-by: Danil <deniallugo@gmail.com>
Co-authored-by: Lyova Potyomkin <lyova.potyomkin@gmail.com>
Co-authored-by: kelemeno <kl@matterlabs.dev>
Co-authored-by: Stanislav Bezkorovainyi <stanislavbezkor@gmail.com>
Co-authored-by: Raid Ateir <ra@matterlabs.dev>
Co-authored-by: kelemeno <34402761+kelemeno@users.noreply.github.com>
Co-authored-by: Jmunoz <juanmunoz890@gmail.com>
Co-authored-by: Bence Haromi <bence.haromi@gmail.com>
Co-authored-by: Danil <deniallugo@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@StanislavBreadless StanislavBreadless StanislavBreadless approved these changes

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

Successfully merging this pull request may close these issues.
7 participants
@vladbochok @StanislavBreadless @ly0va @kelemeno @juan518munoz @benceharomi @Deniallugo