Skip to content

initial refactoring of (I)Staking and StakingStore #2057

initial refactoring of (I)Staking and StakingStore

initial refactoring of (I)Staking and StakingStore #2057

Re-run triggered September 10, 2024 17:20
Status Failure
Total duration 4m 23s
Artifacts 2

build.yml

on: pull_request
Build and test (Hardhat)
2m 8s
Build and test (Hardhat)
Build contracts (Foundry - default profile with optimizer)
1m 7s
Build contracts (Foundry - default profile with optimizer)
Run tests (Foundry - ci profile)
4m 7s
Run tests (Foundry - ci profile)
Execute test deployment on local chain
1m 32s
Execute test deployment on local chain
Publish npm package to npmjs
0s
Publish npm package to npmjs
Fit to window
Zoom out
Zoom in

Annotations

3 errors and 10 warnings
Execute test deployment on local chain
Process completed with exit code 1.
Run tests (Foundry - ci profile)
The runner has received a shutdown signal. This can happen when the runner service is stopped, or a manually started runner is canceled.
Run tests (Foundry - ci profile)
Process completed with exit code 143.
Build and test (Hardhat): contracts/accounting/AccountingServiceManager.sol#L13
Explicitly mark visibility in function (Set ignoreConstructors to true if using solidity >=0.7.0)
Build and test (Hardhat): contracts/authorization/AccessAdmin.sol#L25
Function name must be in mixedCase
Build and test (Hardhat): contracts/authorization/AccessAdmin.sol#L26
Function name must be in mixedCase
Build and test (Hardhat): contracts/authorization/AccessAdmin.sol#L34
Contract has 16 states declarations but allowed no more than 15
Build and test (Hardhat): contracts/authorization/AccessAdmin.sol#L120
Function name must be in mixedCase
Build and test (Hardhat): contracts/authorization/Authorization.sol#L26
Explicitly mark visibility in function (Set ignoreConstructors to true if using solidity >=0.7.0)
Build and test (Hardhat): contracts/authorization/IAuthorization.sol#L5
imported name RoleId is not used
Build and test (Hardhat): contracts/authorization/ServiceAuthorization.sol#L32
Immutable variables name are set to be in capitalized SNAKE_CASE
Build and test (Hardhat): contracts/authorization/ServiceAuthorization.sol#L54
Explicitly mark visibility in function (Set ignoreConstructors to true if using solidity >=0.7.0)
Build and test (Hardhat): contracts/distribution/BasicDistribution.sol#L10
imported name NftIdLib is not used