Skip to content

Fix dependency conflicts #233

Fix dependency conflicts

Fix dependency conflicts #233

Triggered via pull request July 8, 2024 12:07
Status Failure
Total duration 55s
Artifacts

eslint.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 4 warnings
eslint: src/edit.js#L4
'lodash' should be listed in the project's dependencies. Run 'npm i -S lodash' to add it
eslint
Process completed with exit code 1.
eslint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
eslint
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v2, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
eslint: src/webamp.js#L85
React Hook useEffect has missing dependencies: 'audio', 'currentSkin', and 'preview'. Either include them or remove the dependency array. Mutable values like 'divRef.current' aren't valid dependencies because mutating them doesn't re-render the component
eslint: src/webamp.js#L107
React Hook useEffect has a missing dependency: 'webamp'. Either include it or remove the dependency array