search

safe-global / safe-smart-account

Safe allows secure management of blockchain assets.
https://safe.global
GNU Lesser General Public License v3.0
1.84k stars 907 forks source link

Adjust 1.4.1 tests for zkSync #625

Open ElvisKrop opened 1 year ago

ElvisKrop commented 1 year ago

As agreed previously https://github.com/safe-global/safe-contracts/pull/588#issuecomment-1613617245

github-actions[bot] commented 1 year ago

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

ElvisKrop commented 1 year ago

I have read the CLA Document and I hereby sign the CLA

nick8319 commented 1 year ago

I have read the CLA Document and I hereby sign the CLA

ElvisKrop commented 1 year ago

Hello @mmv08! I hope you are doing well :relieved: I was able to rebase this branch onto the main with latest changes. However, we cannot merge zksync related changes as zksync packages use ethers-v5, while main branch has some deps that rely on ethers-v6. Please take a look at this issue (https://github.com/matter-labs/hardhat-zksync/issues/311) created by @NomicFoundation in June. It's sad...

I can see only two options here:

  1. If zksync changes must get in the main branch before the next release - we have to downgrade all dependencies back in the repository to use ethers-v5. After @matter-labs migrates all their packages to the ethers-v6 we can migrate too.
  2. Otherwise, we have to wait until @matter-labs will migrate to the ethers-v6. Only then we can come back to this PR and try to make it work.

Both options are not so good from dev perspective. If you have anything in mind, please let us know.