Skip to content

[DPCP-31] Audio Player #20

[DPCP-31] Audio Player

[DPCP-31] Audio Player #20

Triggered via pull request August 4, 2024 18:37
Status Failure
Total duration 48s
Artifacts

pr-checks.yaml

on: pull_request
oneiros::pr_checks::build
30s
oneiros::pr_checks::build
oneiros::pr_checks::lint
29s
oneiros::pr_checks::lint
oneiros::pr_checks::test
38s
oneiros::pr_checks::test
Fit to window
Zoom out
Zoom in

Annotations

12 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
Property 'isPlaying' does not exist on type 'never'.
oneiros::pr_checks::build: src/molecules/02_AudioPlayer/AudioPlayer.tsx#L1
Property 'pause' does not exist on type 'never'.
oneiros::pr_checks::build: src/molecules/02_AudioPlayer/AudioPlayer.tsx#L1
Property 'currentTime' does not exist on type 'never'.
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
'audioElement.current' is possibly 'undefined'.
oneiros::pr_checks::build: src/molecules/02_AudioPlayer/AudioPlayer.tsx#L1
Expected 2 arguments, but got 1.
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
'audioElement.current' is possibly 'undefined'.
oneiros::pr_checks::build: src/molecules/02_AudioPlayer/AudioPlayer.tsx#L1
'audioElement.current' is possibly 'undefined'.
src/molecules/02_AudioPlayer/__test__/02-AudioPlayer.test.tsx > AudioPlayer component > AudioPlayer should render correctly: src/molecules/02_AudioPlayer/AudioPlayer.tsx#L118
TypeError: Cannot read properties of null (reading 'removeEventListener') ❯ src/molecules/02_AudioPlayer/AudioPlayer.tsx:118:28 ❯ safelyCallDestroy node_modules/react-dom/cjs/react-dom.development.js:22971:5 ❯ commitHookEffectListUnmount node_modules/react-dom/cjs/react-dom.development.js:23139:11 ❯ commitPassiveUnmountInsideDeletedTreeOnFiber node_modules/react-dom/cjs/react-dom.development.js:25140:11 ❯ commitPassiveUnmountEffectsInsideOfDeletedTree_begin node_modules/react-dom/cjs/react-dom.development.js:25087:5 ❯ commitPassiveUnmountEffects_begin node_modules/react-dom/cjs/react-dom.development.js:24995:11 ❯ commitPassiveUnmountEffects node_modules/react-dom/cjs/react-dom.development.js:24980:3 ❯ flushPassiveEffectsImpl node_modules/react-dom/cjs/react-dom.development.js:27077:3 ❯ flushPassiveEffects node_modules/react-dom/cjs/react-dom.development.js:27023:14 ❯ commitRootImpl node_modules/react-dom/cjs/react-dom.development.js:26974:5
oneiros::pr_checks::test
Process completed with exit code 1.
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/
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/