Skip to content

Merge pull request #2 from zengzengzenghuy/fix/mismatch_contract #1

Merge pull request #2 from zengzengzenghuy/fix/mismatch_contract

Merge pull request #2 from zengzengzenghuy/fix/mismatch_contract #1

Triggered via push April 16, 2024 10:25
Status Failure
Total duration 9m 55s
Artifacts

main.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

7 errors and 16 warnings
lint: contracts/helpers/AMBBridgeHelper.sol#L1
Compiler version 0.7.6 does not satisfy the 0.4.24 semver requirement
lint: contracts/upgradeable_contracts/BridgeValidators.sol#L6
Replace ⏎········uint256·_requiredSignatures,⏎········address[]·_initialValidators,⏎········address·_owner with uint256·_requiredSignatures,·address[]·_initialValidators,·address·_owner)
lint: contracts/upgradeable_contracts/BridgeValidators.sol#L10
Replace ····)·external·onlyRelevantSender·returns·(bool) with ········external⏎········onlyRelevantSender⏎········returns·(bool)⏎···
lint
Process completed with exit code 1.
coverage
Process completed with exit code 1.
test
Process completed with exit code 1.
gasreport
Process completed with exit code 1.
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v1, actions/checkout@v2, actions/cache@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1, actions/checkout@v2, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
lint: contracts/gsn/token_paymaster/IUniswapV2Router02.sol#L6
Function name must be in mixedCase
lint: contracts/helpers/AMBBridgeHelper.sol#L28
Explicitly mark visibility of state
lint: contracts/helpers/AMBBridgeHelper.sol#L29
Variable name must be in mixedCase
lint: contracts/helpers/AMBBridgeHelper.sol#L31
Explicitly mark visibility in function
lint: contracts/mocks/UniswapRouterMock.sol#L8
Function name must be in mixedCase
lint: contracts/mocks/UniswapRouterMock.sol#L30
Avoid to use ".call.value()()"
lint: contracts/PermittableToken.sol#L9
Variable name must be in mixedCase
lint: contracts/upgradeable_contracts/arbitrary_message/HomeAMB.sol#L23
Function name must be in mixedCase
coverage
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v1, actions/checkout@v2, actions/cache@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
coverage
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1, actions/checkout@v2, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v1, actions/checkout@v2, actions/cache@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1, actions/checkout@v2, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
gasreport
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v1, actions/checkout@v2, actions/cache@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
gasreport
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v1, actions/checkout@v2, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/