Skip to content

slither doc, feecalc fixes, add mock deposit contract and feecalc to … #65

slither doc, feecalc fixes, add mock deposit contract and feecalc to …

slither doc, feecalc fixes, add mock deposit contract and feecalc to … #65

Triggered via pull request June 28, 2024 01:14
Status Success
Total duration 1m 19s
Artifacts

node.js.yml

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

Annotations

4 errors and 5 warnings
build (20.x): contracts/v2/periphery/ETHDepositGaurd.sol#L9
Compiler version must be declared
build (20.x): contracts/v2/periphery/ETHDepositGaurd.sol#L10
Constructor keyword not available before 0.4.22 (undefined)
build (20.x): contracts/v2/periphery/FeeCalc.sol#L44
Variable "sender" is unused
build (20.x): contracts/v2/periphery/FeeCalc.sol#L52
Variable "sender" is unused
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/WSGEth.sol#L21
Avoid making time-based decisions in your business logic
build (20.x): contracts/v2/periphery/ETHDepositGaurd.sol#L10
Code contains empty blocks
build (20.x): contracts/v2/periphery/Zap.sol#L12
Variable name must be in mixedCase
build (20.x): contracts/v2/periphery/Zap.sol#L18
Variable name must be in mixedCase