Skip to content
Lint pull requests

fix BB-343 : Integrated the modification of identifiers and aliases into the primary editing workflow #1033

Sign in to view logs

fix BB-343 : Integrated the modification of identifiers and aliases into the primary editing workflow

fix BB-343 : Integrated the modification of identifiers and aliases into the primary editing workflow #1033

Triggered via pull request April 24, 2024 14:40
@Tarunmeena0901Tarunmeena0901
synchronize #1043
Status Success
Total duration 51s
Artifacts –

lint-pull-requests.yml

on: pull_request_target
Run linters for pull requests
41s
Run linters for pull requests
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 5 warnings
test/src/api/routes/test-author.js ► Browse Author ► "before all" hook for "should throw an error if trying to browse more than one entity": test-results.json#L0
Failed test found in: test-results.json Error: insert into "bookbrainz"."editor" ("cached_metabrainz_name", "gender_id", "id", "metabrainz_user_id", "name", "privs", "revisions_applied", "type_id") values ($1, $2, $3, $4, $5, $6, $7, $8) returning * - duplicate key value violates unique constraint "editor_pkey"
test/src/server/routes/merge.js ► Merge routes with entity editing priv /cancel route ► "after all" hook for "should clear the merge queue": test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/merge.js)
test/src/server/routes/merge.js ► Merge routes with entity editing priv /cancel route ► should clear the merge queue: test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/merge.js)
test/src/server/routes/relationship.js ► Relationship ► "after all" hook for "should be able to add new relationships without having previous relationships": test-results.json#L0
Failed test found in: test-results.json Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/bookbrainz-site/bookbrainz-site/test/src/server/routes/relationship.js)
src/client/components/pages/revision.js#L177
Unexpected 'todo' comment: 'TODO: Add proper error handling.' (no-warning-comments)
src/server/helpers/collectionRouteUtils.js#L22
'log' is defined but never used (@typescript-eslint/no-unused-vars)
src/server/routes/register.js#L22
'handler' is defined but never used (@typescript-eslint/no-unused-vars)
Run linters for pull requests
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, wearerequired/lint-action@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Run linters for pull requests
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, wearerequired/lint-action@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/