[DPCP-31] Audio Player #29
pr-checks.yaml
on: pull_request
oneiros::pr_checks::build
36s
oneiros::pr_checks::lint
32s
oneiros::pr_checks::test
33s
Annotations
5 errors and 13 warnings
oneiros::pr_checks::build:
src/molecules/02_AudioPlayer/AudioPlayer.tsx#L1
'audioElement.current' is possibly 'undefined'.
|
oneiros::pr_checks::build:
src/molecules/02_AudioPlayer/AudioPlayer.tsx#L1
Type 'string | null | undefined' is not assignable to type 'string | undefined'.
|
oneiros::pr_checks::build:
src/molecules/02_AudioPlayer/AudioPlayer.tsx#L1
Type 'MutableRefObject<HTMLAudioElement | undefined>' is not assignable to type 'LegacyRef<HTMLAudioElement> | undefined'.
|
oneiros::pr_checks::build:
src/molecules/02_AudioPlayer/AudioPlayer.tsx#L1
Type 'boolean' is not assignable to type 'string'.
|
oneiros::pr_checks::build
Process completed with exit code 2.
|
oneiros::pr_checks::test
The following actions uses node12 which is deprecated and will be forced to run on node16: 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/
|
oneiros::pr_checks::test
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, 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/
|
oneiros::pr_checks::lint
The following actions uses node12 which is deprecated and will be forced to run on node16: 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/
|
oneiros::pr_checks::lint
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, 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/
|
oneiros::pr_checks::lint:
src/atoms/03_Link/Link.tsx#L60
The 'isInternal' function makes the dependencies of useEffect Hook (at line 114) change on every render. To fix this, wrap the definition of 'isInternal' in its own useCallback() Hook
|
oneiros::pr_checks::lint:
src/atoms/03_Link/Link.tsx#L67
The 'toProtocol' function makes the dependencies of useEffect Hook (at line 114) change on every render. To fix this, wrap the definition of 'toProtocol' in its own useCallback() Hook
|
oneiros::pr_checks::lint:
src/atoms/05_SystemIcon/SystemIcon.tsx#L91
React Hook useMemo has an unnecessary dependency: 'theme'. Either exclude it or remove the dependency array
|
oneiros::pr_checks::lint:
src/atoms/09_TextInput/TextInput.tsx#L172
React Hook useMemo has an unnecessary dependency: 'extraProps.value'. Either exclude it or remove the dependency array
|
oneiros::pr_checks::lint:
src/atoms/12_DateRange/DateRange.tsx#L121
React Hook useEffect has a missing dependency: 'handleClear'. Either include it or remove the dependency array
|
oneiros::pr_checks::lint:
src/molecules/02_AudioPlayer/AudioPlayer.tsx#L119
The ref value 'audioElement.current' will likely have changed by the time this effect cleanup function runs. If this ref points to a node rendered by React, copy 'audioElement.current' to a variable inside the effect, and use that variable in the cleanup function
|
oneiros::pr_checks::lint:
src/molecules/02_AudioPlayer/AudioPlayer.tsx#L121
React Hook useEffect has a missing dependency: 'handleStatus'. Either include it or remove the dependency array
|
oneiros::pr_checks::build
The following actions uses node12 which is deprecated and will be forced to run on node16: 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/
|
oneiros::pr_checks::build
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, 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/
|