Skip to content

[STABLE-7559]: Migrate from Slither to Mythril for static analysis (#51) #60

[STABLE-7559]: Migrate from Slither to Mythril for static analysis (#51)

[STABLE-7559]: Migrate from Slither to Mythril for static analysis (#51) #60

Triggered via push November 21, 2024 19:41
Status Success
Total duration 45m 41s
Artifacts

ci.yml

on: push
analyze-message-transmitter
45m 32s
analyze-message-transmitter
analyze-token-messenger-minter
13m 56s
analyze-token-messenger-minter
release-sbom  /  release_attach_assets
8s
release-sbom / release_attach_assets
scan  /  scan
scan / scan
Fit to window
Zoom out
Zoom in

Annotations

10 warnings and 1 notice
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)