Merge main back into protocol defense #3785
Annotations
10 errors and 1 warning
|
Lint:
l1-contracts/contracts/bridge/L1SharedBridge.sol#L133
GC: Use Custom Errors instead of require statements
|
|
Lint:
l1-contracts/contracts/bridge/L1SharedBridge.sol#L177
GC: Use Custom Errors instead of require statements
|
|
Lint:
l1-contracts/contracts/bridge/L1SharedBridge.sol#L279
GC: Use Custom Errors instead of require statements
|
|
Lint:
l1-contracts/contracts/bridge/L1SharedBridge.sol#L280
GC: Use Custom Errors instead of require statements
|
|
Lint:
l1-contracts/contracts/bridge/L1SharedBridge.sol#L290
GC: Use Custom Errors instead of require statements
|
|
Lint:
l1-contracts/deploy-scripts/dev/SetupLegacyBridge.s.sol#L26
GC: For [ Addresses ] struct, packing seems inefficient. Try rearranging to achieve 32bytes slots
|
|
Lint:
l1-contracts/deploy-scripts/Utils.sol#L394
GC: For [ i ] variable, increment/decrement by 1 using: [ ++variable ] to save gas
|
|
Lint:
l1-contracts/deploy-scripts/Utils.sol#L399
GC: For [ i ] variable, increment/decrement by 1 using: [ ++variable ] to save gas
|
|
Lint:
l1-contracts/deploy-scripts/Utils.sol#L420
GC: For [ i ] variable, increment/decrement by 1 using: [ ++variable ] to save gas
|
|
Lint:
l1-contracts/deploy-scripts/Utils.sol#L425
GC: Found [ .length ] property in Loop condition. Suggestion: assign it to a variable
|
|
The following actions use a deprecated Node.js version and will be forced to run on node20: 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/
|
Loading