You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
smol-ninja
changed the title
Change vm visibility to internal in StdAssertions
Change vm visibility to internal in StdAssertions and StdUtilsMar 25, 2024
This will cause a solc Identifer already declared error because we define vm as internal in CommonBase which is used by Test and Script. If you don't want to inherit from Test and Script you can instead inherit from CommonBase
Currently, the visibility of
vm
inStdAssertions
andStdUtils
is private which means it cannot be use by the derived contract.forge-std/src/StdAssertions.sol
Line 8 in bb4ceea
forge-std/src/StdUtils.sol
Line 17 in bb4ceea
If we change it to internal,
vm
can be used directly with StdAssertions and StdUtils imports.The text was updated successfully, but these errors were encountered: