Upgrade npm #1423
kpshervaAnnotations
1 error and 12 warnings
|
Tests (20.x)
Process completed with exit code 1.
|
|
Tests (20.x)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Tests (20.x)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v2, actions/setup-node@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Tests (20.x):
src/lib/authentication/state/reducer.js#L17
Assign arrow function to a variable before exporting as module default
|
|
Tests (20.x):
src/lib/components/Pagination/Pagination.js#L47
propType "currentSize" is not required, but has no corresponding defaultProps declaration
|
|
Tests (20.x):
src/lib/components/Pagination/Pagination.js#L48
propType "totalPages" is not required, but has no corresponding defaultProps declaration
|
|
Tests (20.x):
src/lib/components/Pagination/Pagination.js#L51
propType "loading" is not required, but has no corresponding defaultProps declaration
|
|
Tests (20.x):
src/lib/components/Pagination/Pagination.js#L52
propType "firstItem" is not required, but has no corresponding defaultProps declaration
|
|
Tests (20.x):
src/lib/components/Pagination/Pagination.js#L53
propType "lastItem" is not required, but has no corresponding defaultProps declaration
|
|
Tests (20.x):
src/lib/components/SearchBar/SearchBarILS.js#L8
inputRef should be placed after componentDidMount
|
|
Tests (20.x):
src/lib/components/backoffice/DeleteRecordModal/state/reducer.js#L10
Assign arrow function to a variable before exporting as module default
|
|
Tests (20.x):
src/lib/components/backoffice/UrlList/UrlList.js#L15
Do not use Array index in keys
|
|
Tests (20.x):
src/lib/components/backoffice/buttons/ViewDetailsButtons/PatronDetailsLink.js#L18
Fragments should contain more than one child - otherwise, there’s no need for a Fragment at all
|