Skip to content

Merge pull request #14 from chimera-defi/ci #3

Merge pull request #14 from chimera-defi/ci

Merge pull request #14 from chimera-defi/ci #3

Triggered via push May 30, 2024 18:06
Status Success
Total duration 55s
Artifacts

node.js.yml

on: push
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
build (20.x)
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/.
build (20.x): contracts/v2/core/RewardsReceiver.sol#L20
Immutable variables name are set to be in capitalized SNAKE_CASE
build (20.x): contracts/v2/core/Rollover.sol#L21
Immutable variables name are set to be in capitalized SNAKE_CASE
build (20.x): contracts/v2/core/SgETH.sol#L4
imported name ERC20 is not used
build (20.x): contracts/v2/core/SgETH.sol#L14
GC: Use Custom Errors instead of require statements
build (20.x): contracts/v2/core/SharedDepositMinterV2.sol#L65
Function order is incorrect, state variable declaration can not go after custom error definition (line 63)
build (20.x): contracts/v2/core/SharedDepositMinterV2.sol#L65
Immutable variables name are set to be in capitalized SNAKE_CASE
build (20.x): contracts/v2/core/SharedDepositMinterV2.sol#L66
Immutable variables name are set to be in capitalized SNAKE_CASE
build (20.x): contracts/v2/core/SharedDepositMinterV2.sol#L204
GC: Use Custom Errors instead of require statements
build (20.x): contracts/v2/core/Withdrawals.sol#L4
imported name SafeERC20 is not used
build (20.x): contracts/v2/core/Withdrawals.sol#L4
imported name SafeMath is not used