Create python-app.yml #1
Triggered via pull request
September 27, 2024 21:44
Status
Failure
Total duration
3m 53s
Artifacts
–
ci.yml
on: pull_request
lint-and-test
3m 32s
scan
/
scan
6s
release-sbom
/
release_attach_assets
Annotations
1 error and 11 warnings
scan / scan
Dependency review is not supported on this repository. Please ensure that Dependency graph is enabled along with GitHub Advanced Security on private repositories, see https://github.com/ysl33p/evm-cctp-contracts/settings/security_analysis
|
lint-and-test
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/
|
lint-and-test:
anvil/Counter.sol#L6
Explicitly mark visibility in function (Set ignoreConstructors to true if using solidity >=0.7.0)
|
lint-and-test:
anvil/scripts/Counter.s.sol#L7
Code contains empty blocks
|
lint-and-test:
scripts/deploy.s.sol#L9
Contract has 20 states declarations but allowed no more than 15
|
lint-and-test:
src/interfaces/IMessageTransmitter.sol#L25
Code contains empty blocks
|
lint-and-test:
src/MessageTransmitter.sol#L88
Explicitly mark visibility in function (Set ignoreConstructors to true if using solidity >=0.7.0)
|
lint-and-test:
src/MessageTransmitter.sol#L157
Error message for require is too long
|
lint-and-test:
src/MessageTransmitter.sol#L164
Error message for require is too long
|
lint-and-test:
src/MessageTransmitter.sol#L202
Error message for require is too long
|
lint-and-test:
src/roles/Attestable.sol#L64
Constant name must be in capitalized SNAKE_CASE
|
lint-and-test:
src/roles/Attestable.sol#L87
Explicitly mark visibility in function (Set ignoreConstructors to true if using solidity >=0.7.0)
|