Skip to content

remove unnecessary change #52100

remove unnecessary change

remove unnecessary change #52100

Triggered via push January 24, 2024 22:47
Status Success
Total duration 3m 59s
Artifacts

solidity.yml

on: push
Detect changes
6s
Detect changes
Prepublish Test
1m 16s
Prepublish Test
Native Compilation
3m 34s
Native Compilation
Solidity Lint
38s
Solidity Lint
Prettier Formatting
39s
Prettier Formatting
Fit to window
Zoom out
Zoom in

Annotations

1 error and 25 warnings
Solidity Lint
Got error Error: No job for job name: "Lint" found during self-lookup, invalid job name given? Available jobs names + ids: Detect changes|20837703485,Solidity Lint|20837709488,Native Compilation|20837709690,Prepublish Test|20837709885,Prettier Formatting|20837710072 , failing action
Detect changes
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: dorny/paths-filter@4512585405083f25c027a35db413c2b3b9006d50. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Solidity Lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: pnpm/action-setup@c3b53f6a16e57305370b4ae5a540c2077a1d50dd. 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/KeeperRegistrar.test.ts#L37
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Prettier Formatting
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: pnpm/action-setup@c3b53f6a16e57305370b4ae5a540c2077a1d50dd. 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/VRFSubscriptionBalanceMonitor.test.ts#L282
'tx' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/ERC20BalanceMonitor.test.ts#L276
'tx' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/EthBalanceMonitor.test.ts#L267
'tx' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/KeeperRegistrar2_0.test.ts#L40
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/automation/KeeperRegistrar2_0.test.ts#L42
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/automation/KeeperRegistrar2_0.test.ts#L46
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/automation/KeeperRegistry1_2.test.ts#L77
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/automation/KeeperRegistry1_2.test.ts#L1094
'maxPayment' is never reassigned. Use 'const' instead
Solidity Lint
When fallback is not payable you will not be able to receive ether
Solidity Lint
global import of path ../../shared/access/ConfirmedOwner.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
Solidity Lint
global import of path ../interfaces/AutomationCompatibleInterface.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
Solidity Lint
global import of path ../interfaces/StreamsLookupCompatibleInterface.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
Solidity Lint
global import of path ../../ChainSpecificUtil.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
Solidity Lint
Constant variable c_feedParamKey is not in all caps, it should be C_FEEDPARAMKEY
Solidity Lint
Explicitly mark visibility of state
Solidity Lint
Constant variable c_timeParamKey is not in all caps, it should be C_TIMEPARAMKEY
Solidity Lint
Explicitly mark visibility of state
Solidity Lint
Constant variable scale is not in all caps, it should be SCALE
Prepublish Test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: pnpm/action-setup@c3b53f6a16e57305370b4ae5a540c2077a1d50dd. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Native Compilation
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: pnpm/action-setup@c3b53f6a16e57305370b4ae5a540c2077a1d50dd, actions/setup-go@v4, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.