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

[MULTI ZNS] MAIN #121

Open
wants to merge 125 commits into
base: development
Choose a base branch
from
Open

[MULTI ZNS] MAIN #121

wants to merge 125 commits into from

Conversation

Whytecrowe
Copy link
Collaborator

@Whytecrowe Whytecrowe commented Dec 17, 2024 •
edited
Loading

Main Branch for the 2nd release of ZNS that works cross-chain.
ZNS Ethereum will be upgraded based on this code and ZNS ZChain will be deployed from this code.
All PRs that work on the new ZNS for Mainnet will be merged here.

DO NOT MERGE THIS PR!

MichaelKorchagin and others added 30 commits March 16, 2024 16:21
@MichaelKorchagin
String Resolver init
3662fac
@MichaelKorchagin
SR mission added
9f774a9
@MichaelKorchagin
Tests for SR
30d6e95
@MichaelKorchagin
Add DB drop to test, fix naming
ef3b152
@MichaelKorchagin
Linter fixes
a237e61
@MichaelKorchagin
Adjusting the deployment process. Сorrections based on review
b625249
@MichaelKorchagin
Operator initialize
e3ee2c3
@MichaelKorchagin
Only method deletion
09fed3b
@MichaelKorchagin
Replacing require with a custom error. New names of variables in stri…
96de554 
…ngs mapping
@Whytecrowe
update solidity version in all contracts
bd583d5
@Whytecrowe
update all the possible packages, including zToken
e7cad5f
@Whytecrowe
rework meow token deploy logic to use imported artifacts and it's moc…
7e127b4 
…k to just be a generic erc20 because of issues with nested deps in hardhat
@Whytecrowe
fix inheritance issues in DomainToken
0658ca8
@Whytecrowe
fix hardhat config to use proper solc versions
8b9a029
@Whytecrowe
fix all tests based on the new changes to revert errors and other things
7eb8f1a
@Whytecrowe
try newer node image for CI
40963a7
@Whytecrowe
try adding more memory to hardhat coverage
17b764f
@Whytecrowe
try a different way
95bbb0e
@Whytecrowe
try exporting old space size for Node before running coverage
e4f626e
@Whytecrowe
remove redundant imports
6561363
@Whytecrowe
convert requires to custom errors for Access and Pricers
70a983c
@Whytecrowe
make errors for Registry and Registrars
f91e990
@Whytecrowe
add errors for AddressResolver and Treasury
b65b7f5
@Whytecrowe
try rolling back solcover version
aa73a8b
@Whytecrowe
update yarn.lock
03f6d7c
@Whytecrowe
fix contract errors to compile
11ea094
@Whytecrowe
fix errors in if statements, add string utils
c3c9312
@Whytecrowe
fix error in StringUtils
e9dd200
@Whytecrowe
fix error constants in TS
63fd27a
@Whytecrowe
first pass on reworking test error checks
de901cf
JamesEarle and others added 18 commits September 26, 2024 13:30
@JamesEarle
update cpt eslint config
2661eae
@JamesEarle
update yarn lock
ab0286b
@JamesEarle
Updated config.yml
9bc171a
@JamesEarle
Updated config.yml
511b248
@JamesEarle
update eslint with new jsdoc rule turned off
d5ee145
@JamesEarle
Merge branch 'feat/modify-transfers' of https://github.com/zer0-os/zNS …
203b90d 
…into feat/modify-transfers
@JamesEarle
remove tenderly setup function in HH config
d6f3763
@MichaelKorchagin
Review fixes. Config changed. New config build. Mission for ZToken ch…
7ecd641 
…anged. Deleted redundant approvals. Mock token fix.
@MichaelKorchagin
Type fixes.
76182b3
@JamesEarle
add tests for setRegistry in DmoainToken
e9015ac
@JamesEarle
simplify import from helpers
fbe4504
@JamesEarle
reverse change to node image in circleci build
5831357
@JamesEarle
Updated config.yml
89232f2
@JamesEarle
Updated config.yml
dd8c2a7
@JamesEarle
Updated config.yml
46fd565
@JamesEarle
lint modified import list
ddcdb97
@JamesEarle
remove custom auth in circleCI, update func name in sol, add extra te…
96e6cc0 
…st for use of safeTransferFrom checks
@Whytecrowe
Merge branch 'rc/zns-meowchain' into rc/multi-zns-main
ab63377
@openzeppelin-code OpenZeppelin Code
Copy link

openzeppelin-code bot commented Dec 17, 2024 •
edited
Loading

[MULTI ZNS] MAIN

Generated at commit: 1c9d77520d566dae21688b143484ab08f27110ab

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total		 1
1
0
2
17
21
Dependencies Critical
High
Medium
Low
Note
Total		 0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

@codecov Codecov
Copy link

codecov bot commented Dec 17, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 99.29577% with 1 line in your changes missing coverage. Please review.

Project coverage is 99.82%. Comparing base (5dd6a86) to head (1c9d775).

Additional details and impacted files 
@@               Coverage Diff               @@
##           development     #121      +/-   ##
===============================================
+ Coverage        99.80%   99.82%   +0.02%     
===============================================
  Files               11       12       +1     
  Lines              512      576      +64     
  Branches           127      129       +2     
===============================================
+ Hits               511      575      +64     
  Misses               1        1

This was referenced Dec 17, 2024
Open
Open
@Whytecrowe Whytecrowe mentioned this pull request Dec 17, 2024
Open
@Whytecrowe Whytecrowe mentioned this pull request Dec 18, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Whytecrowe @MichaelKorchagin @JamesEarle