NexusMutual / smart-contracts

GNU General Public License v3.0
170 stars 62 forks source link

feat: Add create2 library linking #1220

Closed rackstar closed 2 months ago

rackstar commented 3 months ago

Context

Since we introduced an external library, we now need to add library linking steps in both find-salt and deploy create2 scripts.

Changes proposed in this pull request

Test plan

Add validation checks on the scripts and tested manually for StakingPool.sol find-salt

Checklist

Review

When reviewing a PR, please indicate intention in comments using the following emojis:

rackstar commented 2 months ago

tenderly fork test:

manually deployed these contracts:

Testing: For some reason the tx failed (maybe my input was wrong) but the simulated tx shows Cover successfully calls StakingPool and then StakingPool successfully calls StakingPoolExtrasLib which indicates the library linking worked: