Skip to content
Linting

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

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() #1085

Triggered via pull request October 27, 2023 02:43
@ItsANameTooItsANameToo
synchronize #287
fix/api-sync/ignore-constraint-violations
Status Success
Total duration 1m 51s
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(…)`