safe-contracts

Making contracts `memory-safe` #1837

Triggered via pull request December 4, 2023 11:01

remedcu

opened #711

optimizer-enabled

Status Failure

Total duration 1m 12s

Artifacts –

ci.yml

on: pull_request

Matrix: tests

Matrix: benchmarks

2 errors and 24 warnings

Process completed with exit code 1.

Process completed with exit code 1.

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@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-solidity: contracts/common/SecuredTokenTransfer.sol#L23

Avoid to use inline assembly. It is acceptable only in rare cases

lint-solidity: contracts/libraries/Safe130To141Migration.sol#L130

Avoid to use inline assembly. It is acceptable only in rare cases

lint-solidity: contracts/libraries/Safe150Migration.sol#L165

Avoid to use inline assembly. It is acceptable only in rare cases

lint-solidity: contracts/libraries/Safe150Migration.sol#L182

Avoid to use inline assembly. It is acceptable only in rare cases

lint-solidity: contracts/libraries/SafeToL2Migration.sol#L171

Avoid to use inline assembly. It is acceptable only in rare cases

lint-solidity: contracts/proxies/SafeProxy.sol#L36

Avoid to use inline assembly. It is acceptable only in rare cases

lint-solidity: contracts/test/DelegateCaller.sol#L18

Avoid to use inline assembly. It is acceptable only in rare cases

lint-solidity: contracts/test/TestImports.sol#L8

imported name UpgradeableProxy is not used

lint-typescript: src/utils/execution.ts#L178