Merge pull request #96 from RootstockCollective/dao-850 #330
jessgusclarkAnnotations
10 warnings
|
test:
test/upgradeStRIFToken.test.ts#L13
Delete `·`
|
|
test:
contracts/DaoTimelockUpgradableRootstockCollective.sol#L3
global import of path @openzeppelin/contracts-upgradeable/proxy/utils/UUPSUpgradeable.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
|
test:
contracts/DaoTimelockUpgradableRootstockCollective.sol#L4
global import of path @openzeppelin/contracts-upgradeable/governance/TimelockControllerUpgradeable.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
|
test:
contracts/DaoTimelockUpgradableRootstockCollective.sol#L18
Code contains empty blocks
|
|
test:
contracts/EarlyAdoptersRootstockCollective.sol#L195
Avoid to use tx.origin
|
|
test:
contracts/EarlyAdoptersRootstockCollective.sol#L228
Code contains empty blocks
|
|
test:
contracts/exploit/NFTAttacker.sol#L2
Found more than One contract per file. 2 contracts found!
|
|
test:
contracts/exploit/NFTAttacker.sol#L28
GC: Use Custom Errors instead of require statements
|
|
test:
contracts/exploit/NFTAttacker.sol#L74
GC: Use Custom Errors instead of require statements
|
|
test:
contracts/GovernorRootstockCollective.sol#L32
Constant name must be in capitalized SNAKE_CASE
|