Added test for RewardsReceiver and SharedDepositMinterV2 #8
Annotations
1 error and 11 warnings
|
|
|
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
contracts/v2/core/RewardsReceiver.sol#L20
Immutable variables name are set to be in capitalized SNAKE_CASE
|
|
contracts/v2/core/Rollover.sol#L21
Immutable variables name are set to be in capitalized SNAKE_CASE
|
|
contracts/v2/core/SgETH.sol#L4
imported name ERC20 is not used
|
|
contracts/v2/core/SgETH.sol#L14
GC: Use Custom Errors instead of require statements
|
|
contracts/v2/core/SharedDepositMinterV2.sol#L65
Function order is incorrect, state variable declaration can not go after custom error definition (line 63)
|
|
contracts/v2/core/SharedDepositMinterV2.sol#L65
Immutable variables name are set to be in capitalized SNAKE_CASE
|
|
contracts/v2/core/SharedDepositMinterV2.sol#L66
Immutable variables name are set to be in capitalized SNAKE_CASE
|
|
contracts/v2/core/SharedDepositMinterV2.sol#L204
GC: Use Custom Errors instead of require statements
|
|
contracts/v2/core/Withdrawals.sol#L4
imported name SafeERC20 is not used
|
|
contracts/v2/core/Withdrawals.sol#L4
imported name SafeMath is not used
|
This job failed
Loading