Cyfrin / foundry-defi-stablecoin-cu

250 stars 117 forks source link

Update Config to Use the Latest Open Zeppelin ERC20Mock #7

Closed k-g-j closed 1 year ago

k-g-j commented 1 year ago

Hello, I noticed you were using an older version of the ERC20Mock in your HelperConfig.s.sol script. The version you are using seems to be updated 2 years ago (6/7/2021). The latest ERC20Mock as of the update on 5/29/2023 requires no constructor arguments. You may want to change the config script to keep up to date and so people don't run into issues was installing the latest Open Zeppelin contracts lib.

PatrickAlphaC commented 1 year ago

hmmmmmmmm. Thank you! Looks like I wrote the code before this update rolled in...

If they follow along with the video and choose the version I use this should be fine. Going to close this for now, but let's see how many people run into this issue.