Skip to content

chore: add solhint #161

chore: add solhint

chore: add solhint #161

Triggered via pull request October 16, 2023 10:06
Status Failure
Total duration 1m 13s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

test.yml

on: pull_request
security  /  Check security issues
24s
security / Check security issues
actions  /  Check GitHub Actions issues
7s
actions / Check GitHub Actions issues
Linters
1m 3s
Linters
Foundry project
29s
Foundry project
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 10 warnings
Linters: src/CommunityStakingBondManager.sol#L51
Declaration of constant PENALIZE_BOND_ROLE should go before errors definitions
Linters: src/CommunityStakingBondManager.sol#L55
'bondShares' should start with underscore
Linters: src/CommunityStakingBondManager.sol#L69
'_commonBondSize' should not start with underscore
Linters: src/CommunityStakingBondManager.sol#L70
'_admin' should not start with underscore
Linters: src/CommunityStakingBondManager.sol#L71
'_lidoLocator' should not start with underscore
Linters: src/CommunityStakingBondManager.sol#L72
'_wstETH' should not start with underscore
Linters: src/CommunityStakingBondManager.sol#L73
'_communityStakingModule' should not start with underscore
Linters: src/CommunityStakingBondManager.sol#L74
'_penalizeRoleMembers' should not start with underscore
Linters: src/CommunityStakingBondManager.sol#L106
'_fdAddress' should not start with underscore
Linters: src/CommunityStakingBondManager.sol#L286
'_permit' should not start with underscore
Linters: src/CommunityStakingBondManager.sol#L4
Found more than One contract per file. 2 contracts found!
Linters: src/CommunityStakingBondManager.sol#L53
Variable name must be in mixedCase
Linters: src/CommunityStakingBondManager.sol#L77
Use Custom Errors instead of require statements
Linters: src/CommunityStakingBondManager.sol#L78
Use Custom Errors instead of require statements
Linters: src/CommunityStakingBondManager.sol#L79
Use Custom Errors instead of require statements
Linters: src/CommunityStakingBondManager.sol#L83
Use Custom Errors instead of require statements
Linters: src/CommunityStakingBondManager.sol#L84
Use Custom Errors instead of require statements
Linters: src/CommunityStakingBondManager.sol#L91
Use Custom Errors instead of require statements
Linters: src/CommunityStakingBondManager.sol#L206
Use Custom Errors instead of require statements
Linters: src/CommunityStakingBondManager.sol#L239
Use Custom Errors instead of require statements