Merge pull request #92 from VenusProtocol/develop #351
Annotations
12 warnings
|
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
contracts/Interfaces/IRiskFund.sol#L2
Found more than One contract per file. 2 contracts found!
|
|
contracts/ProtocolReserve/ProtocolShareReserve.sol#L54
Immutable variables name are set to be in capitalized SNAKE_CASE
|
|
contracts/ProtocolReserve/RiskFundStorage.sol#L2
Found more than One contract per file. 4 contracts found!
|
|
contracts/Test/imports.sol#L10
imported name UpgradeableBeacon is not used
|
|
contracts/Test/imports.sol#L11
imported name BeaconProxy is not used
|
|
contracts/Test/imports.sol#L12
imported name ProxyAdmin is not used
|
|
contracts/Test/Mocks/MockConverter.sol#L33
Function name must be in mixedCase
|
|
contracts/Test/Mocks/MockDeflationaryToken.sol#L22
Avoid to use inline assembly. It is acceptable only in rare cases
|
|
contracts/Test/Mocks/MockDeflationaryToken.sol#L68
Use Custom Errors instead of require statements
|
|
contracts/Test/Mocks/MockDeflationaryToken.sol#L77
Use Custom Errors instead of require statements
|
This job succeeded
Loading