Skip to content

Post merge fixes

Post merge fixes #5450

Triggered via push April 8, 2024 15:32
Status Success
Total duration 4m 53s
Artifacts

solidity.yml

on: push
Detect changes
8s
Detect changes
Initialize
9s
Initialize
Tag Check
5s
Tag Check
Solidity Lint
42s
Solidity Lint
Prettier Formatting
55s
Prettier Formatting
Native Compilation
4m 14s
Native Compilation
Publish Beta NPM
0s
Publish Beta NPM
Publish Prod NPM
0s
Publish Prod NPM
Fit to window
Zoom out
Zoom in

Annotations

21 warnings
Initialize
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@93ea575cb5d8a053eaa0ac8fa3b40d7e05a33cc8. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Solidity Lint: contracts/test/v0.8/Cron.test.ts#L43
'libType' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/L2EP/ArbitrumCrossDomainGovernor.test.ts#L331
'dataBuffer' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/L2EP/ArbitrumCrossDomainGovernor.test.ts#L334
'encoded' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/L2EP/ArbitrumCrossDomainGovernor.test.ts#L352
'transaction' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/L2EP/ArbitrumValidator.test.ts#L10
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Solidity Lint: contracts/test/v0.8/L2EP/ArbitrumValidator.test.ts#L11
'arbitrumSequencerStatusRecorderAbi' is defined but never used. Allowed unused vars must match /^_/u
Solidity Lint: contracts/test/v0.8/L2EP/ArbitrumValidator.test.ts#L12
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Solidity Lint: contracts/test/v0.8/L2EP/ArbitrumValidator.test.ts#L13
'arbitrumInboxAbi' is defined but never used. Allowed unused vars must match /^_/u
Solidity Lint: contracts/test/v0.8/L2EP/ArbitrumValidator.test.ts#L14
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Solidity Lint: contracts/test/v0.8/L2EP/OptimismCrossDomainGovernor.test.ts#L375
'dataBuffer' is never reassigned. Use 'const' instead
Solidity Lint
Interface name 'AutomationRegistryBase2_3' must start with "I"
Solidity Lint
GC: For [ StateLegacy ] struct, packing seems inefficient. Try rearranging to achieve 32bytes slots
Solidity Lint
GC: For [ OnchainConfigLegacy ] struct, packing seems inefficient. Try rearranging to achieve 32bytes slots
Solidity Lint
GC: For [ InitialTriggerConfig ] struct, packing seems inefficient. Try rearranging to achieve 32bytes slots
Solidity Lint
Contract has 30 states declarations but allowed no more than 15
Solidity Lint
GC: For [ BillingConfig ] struct, packing seems inefficient. Try rearranging to achieve 32bytes slots
Solidity Lint
Avoid to use tx.origin
Solidity Lint
Avoid to use low level calls
Solidity Lint
Avoid to use low level calls
Solidity Lint
Interface name 'AutomationCompatibleInterface' must start with "I"