Skip to content

PR suggestions

PR suggestions #2

Triggered via push June 12, 2024 10:02
Status Cancelled
Total duration 2m 30s
Artifacts

js-ci.yml

on: push
Check conditional workflows and jobs
9s
Check conditional workflows and jobs
Account UI E2E
0s
Account UI E2E
Matrix: Admin UI E2E
Status Check - Keycloak JavaScript CI
7s
Status Check - Keycloak JavaScript CI
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 24 warnings
Admin UI
Canceling since a higher priority waiting request for 'js-ci-refs/heads/feature_30211-password-age-policy' exists
Build Keycloak
Canceling since a higher priority waiting request for 'js-ci-refs/heads/feature_30211-password-age-policy' exists
Build Keycloak
The operation was canceled.
Status Check - Keycloak JavaScript CI
Process completed with exit code 1.
UI Shared: js/libs/ui-shared/src/user-profile/MultiInputComponent.tsx#L72
React Hook useMemo has a missing dependency: 'defaultValue'. Either include it or remove the dependency array
UI Shared: js/libs/ui-shared/src/user-profile/MultiInputComponent.tsx#L99
React Hook useEffect has a missing dependency: 'name'. Either include it or remove the dependency array
UI Shared: js/libs/ui-shared/src/utils/useStorageItem.ts#L23
React Hook useCallback has missing dependencies: 'keyName' and 'storageArea'. Either include them or remove the dependency array
UI Shared: js/libs/ui-shared/src/utils/useStorageItem.ts#L48
React Hook useEffect has a missing dependency: 'defaultValue'. Either include it or remove the dependency array. If 'setInnerValue' needs the current value of 'defaultValue', you can also switch to useReducer instead of useState and read 'defaultValue' in the reducer
UI Shared: js/libs/ui-shared/src/utils/useStoredState.ts#L34
React Hook useCallback has a missing dependency: 'setStoredValue'. Either include it or remove the dependency array
Account UI: js/apps/account-ui/src/oid4vci/Oid4Vci.tsx#L69
React Hook useEffect has missing dependencies: 'context', 'credentialsIssuer', 'initialSelected', and 'selectOptions'. Either include them or remove the dependency array
Account UI: js/apps/account-ui/src/resources/EditTheResource.tsx#L41
React Hook useEffect has missing dependencies: 'permissions' and 'reset'. Either include them or remove the dependency array
Account UI: js/apps/account-ui/src/resources/ShareTheResource.tsx#L71
React Hook useEffect has a missing dependency: 'append'. Either include it or remove the dependency array
Account UI: js/apps/account-ui/src/utils/usePromise.ts#L73
React Hook useEffect was passed a dependency list that is not an array literal. This means we can't statically verify whether you've passed the correct dependencies
Account UI: js/apps/account-ui/src/utils/usePromise.ts#L73
React Hook useEffect has missing dependencies: 'callback' and 'factory'. Either include them or remove the dependency array. If 'callback' changes too often, find the parent component that defines it and wrap that definition in useCallback
Account UI: js/apps/account-ui/test/applications.spec.ts#L62
Unexpected use of the `.skip()` annotation
Account UI: js/apps/account-ui/test/personal-info/personal-info.spec.ts#L67
Unexpected use of page.waitForSelector()
Account UI: js/apps/account-ui/test/personal-info/personal-info.spec.ts#L81
Unexpected use of page.waitForSelector()
Account UI: js/apps/account-ui/test/personal-info/personal-info.spec.ts#L117
Unexpected use of the `.skip()` annotation
Admin UI: js/apps/admin-ui/cypress/e2e/authentication_test.spec.ts#L84
Unexpected skipped mocha test
Admin UI: js/apps/admin-ui/cypress/e2e/client_scopes_test.spec.ts#L181
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/clients_test.spec.ts#L222
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/clients_test.spec.ts#L232
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/clients_test.spec.ts#L579
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/events_test.spec.ts#L33
Unexpected skipped mocha test
Admin UI: js/apps/admin-ui/cypress/e2e/events_test.spec.ts#L71
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/events_test.spec.ts#L106
Do not wait for arbitrary time periods
Admin UI: js/apps/admin-ui/cypress/e2e/group_test.spec.ts#L264
Unexpected skipped mocha test