Skip to content
Linting

feat(api): add api-transaction-pool package #990

Sign in to view logs

feat(api): add api-transaction-pool package

feat(api): add api-transaction-pool package #990

Triggered via pull request October 6, 2023 08:31
@ItsANameTooItsANameToo
synchronize #268
feat/api/tx-pool
Status Success
Total duration 2m 7s
Artifacts –
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

lint.yml

on: pull_request
Matrix: source
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
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#L87
Use `undefined` instead of `null`
source (20.x): packages/api-common/source/plugins/pagination/extension.ts#L92
Use `undefined` instead of `null`
source (20.x): packages/api-common/source/plugins/pagination/extension.ts#L109
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: `rateLimitResponse`, `rateLimitResult`. A more descriptive name will do too
source (20.x): packages/api-common/source/schemas.ts#L33
Refactor this function to reduce its Cognitive Complexity from 34 to the 15 allowed
source (20.x): packages/api-common/source/schemas.ts#L56
Prefer `.some(…)` over `.find(…)`