Skip to content

feat: vote weighting initial contract #527

feat: vote weighting initial contract

feat: vote weighting initial contract #527

Triggered via pull request March 12, 2024 19:24
Status Failure
Total duration 49s
Artifacts

workflow.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 23 warnings
build: contracts/VoteWeighting.sol#L425
Use double quotes for string literals
build
Process completed with exit code 1.
gitleaks
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-go@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
build
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, 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
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build: contracts/DeploymentFactory.sol#L4
global import of path @openzeppelin/contracts/utils/Create2.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
build: contracts/DeploymentFactory.sol#L5
global import of path ./OLAS.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
build: contracts/DeploymentFactory.sol#L6
global import of path ./veOLAS.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
build: contracts/OLAS.sol#L4
global import of path ../lib/solmate/src/tokens/ERC20.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
build: contracts/OLAS.sol#L28
Immutable variables name are set to be in capitalized SNAKE_CASE
build: contracts/Timelock.sol#L4
global import of path @openzeppelin/contracts/governance/TimelockController.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
build: contracts/VoteWeighting.sol#L4
global import of path ./interfaces/IErrors.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
build: contracts/VoteWeighting.sol#L45
Contract has 16 states declarations but allowed no more than 15
build: contracts/VoteWeighting.sol#L59
Immutable variables name are set to be in capitalized SNAKE_CASE
build: contracts/VoteWeighting.sol#L125
Function name must be in mixedCase
build: scripts/deployment/bridges/optimistic/test/deploy_00_mock_timelock.js#L14
'gasPriceInGwei' is assigned a value but never used
build: scripts/deployment/bridges/polygon/test/fx_goerli_mumbai_hello_world.js#L35
'fxChild' is assigned a value but never used
build: scripts/deployment/bridges/polygon/test/fx_mumbai_goerli_token_transfer.js#L27
'fxRoot' is assigned a value but never used
build: scripts/deployment/bridges/polygon/test/fx_mumbai_goerli_token_transfer.js#L35
'fxChild' is assigned a value but never used
build: scripts/deployment/bridges/polygon/test/fx_mumbai_goerli_token_transfer.js#L51
'bridgedERC20' is assigned a value but never used
build: scripts/deployment/bridges/wormhole/test/deploy_00_mock_timelock.js#L14
'gasPriceInGwei' is assigned a value but never used
build: scripts/proposals/proposal_01_goerli_sync_mainnet.js#L12
'provider' is assigned a value but never used
build: scripts/proposals/proposal_04_CM_guard.js#L11
'timelockAddress' is assigned a value but never used
build: scripts/proposals/proposal_04_CM_guard.js#L23
'CMAddress' is assigned a value but never used
build: scripts/proposals/proposal_04_CM_guard_goerli.js#L11
'timelockAddress' is assigned a value but never used