Linting

feat(api-evm): implement `eth_getTransactionByBlockNumberAndIndex` #2959

Sign in to view logs

Triggered via pull request January 10, 2025 11:03

sebastijankuzner

opened #816

feat/api-evm/eth_getTransactionByBlockNumberAndIndex

Status Success

Total duration 1m 24s

Artifacts –

lint.yml

on: pull_request

Matrix: source

Annotations

11 warnings

source (20.x)

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

source (20.x): packages/api-common/source/controller.ts#L98

'any' overrides all other types in this union type

source (20.x): packages/api-common/source/controller.ts#L98

'any' overrides all other types in this union type

source (20.x): packages/api-common/source/plugins/pagination/extension.ts#L30

Remove this redundant jump

source (20.x): packages/api-common/source/plugins/pagination/extension.ts#L41

Refactor this function to reduce its Cognitive Complexity from 21 to the 15 allowed

source (20.x): packages/api-common/source/plugins/pagination/extension.ts#L72

Use destructured variables over properties

source (20.x): packages/api-common/source/plugins/pagination/extension.ts#L72

Use `undefined` instead of `null`

source (20.x): packages/api-common/source/plugins/pagination/extension.ts#L86

Use `undefined` instead of `null`

source (20.x): packages/api-common/source/plugins/pagination/extension.ts#L91

Use `undefined` instead of `null`

source (20.x): packages/api-common/source/plugins/pagination/extension.ts#L110

Use destructured variables over properties

source (20.x): packages/api-common/source/plugins/rate-limit.ts#L57

Please rename the variable `rateLimitRes`. Suggested names are: `rateLimitResource`, `rateLimitResponse`, `rateLimitResult`. A more descriptive name will do too