feat: add scroll chainId to support list of chainIds · 0xProject/protocol@814596c

Triggered via pull request September 26, 2024 00:55

opened #785

Status Failure

Total duration 5m 31s

Artifacts –

ci.yml

on: pull_request

1 error and 11 warnings

Process completed with exit code 1.

The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

Build and test: contracts/erc20/src/v06/LibERC20TokenV06.sol#L135

Error message for require is too long

Build and test: contracts/erc20/src/v08/LibERC20TokenV08.sol#L131

Error message for require is too long

Build and test: contracts/erc20/src/WETH9.sol#L41

Provide an error message for require

Build and test: contracts/erc20/src/WETH9.sol#L62

Provide an error message for require

Build and test: contracts/erc20/src/WETH9.sol#L65

Provide an error message for require

Build and test: contracts/erc20/test/ZRXTokenTest.t.sol#L25

Explicitly mark visibility of state

Build and test: contracts/governance/src/SecurityCouncil.sol#L38

Error message for revert is too long

Build and test: contracts/governance/src/SecurityCouncil.sol#L72

Error message for require is too long

Build and test: contracts/governance/src/SecurityCouncil.sol#L83

Error message for require is too long

Build and test: contracts/governance/src/ZeroExProtocolGovernor.sol#L37

Explicitly mark visibility in function (Set ignoreConstructors to true if using solidity >=0.7.0)