Skip to content

Scope native and checked type conversion to a single struct and add t… #50997

Scope native and checked type conversion to a single struct and add t…

Scope native and checked type conversion to a single struct and add t… #50997

Triggered via push January 16, 2024 21:25
Status Failure
Total duration 4m 0s
Artifacts

solidity.yml

on: push
Detect changes
8s
Detect changes
Prepublish Test
1m 17s
Prepublish Test
Native Compilation
3m 32s
Native Compilation
Solidity Lint
42s
Solidity Lint
Prettier Formatting
33s
Prettier Formatting
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 20 warnings
Prettier Formatting
Process completed with exit code 1.
Solidity Lint
Process completed with exit code 1.
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
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
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
Solidity Lint
Explicitly mark visibility of state