Skip to content
Linting

fix(api-sync): use insert().orIgnore() instead of calling save() … #1114

Sign in to view logs

fix(api-sync): use insert().orIgnore() instead of calling save() …

fix(api-sync): use insert().orIgnore() instead of calling save() … #1114

Triggered via push November 2, 2023 08:20
@sebastijankuznersebastijankuzner
pushed 8c71449
develop
Status Success
Total duration 1m 44s
Artifacts –
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

lint.yml

on: push
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(…)`