Skip to content

Merge pull request #158 from 10up/dependabot/npm_and_yarn/npm_and_yar… #272

Merge pull request #158 from 10up/dependabot/npm_and_yarn/npm_and_yar…

Merge pull request #158 from 10up/dependabot/npm_and_yarn/npm_and_yar… #272

Triggered via push February 8, 2025 12:52
Status Success
Total duration 42s
Artifacts

eslint.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

2 warnings
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