Newly revamped method for basic visualization #649
ci.yml
on: pull_request
ci
3m 58s
Publish packages to NPM
0s
Annotations
10 errors and 11 warnings
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L312
A function with a name starting with an uppercase letter should only be used as a constructor
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L310
A function with a name starting with an uppercase letter should only be used as a constructor
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L309
Expected '===' and instead saw '=='
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L309
Expected '===' and instead saw '=='
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L76
'pFoSpeed' is never reassigned. Use 'const' instead
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L75
'bFoSpeed' is never reassigned. Use 'const' instead
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L73
'peakFalloff' is never reassigned. Use 'const' instead
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L72
'barFalloff' is never reassigned. Use 'const' instead
|
|
ci:
packages/webamp/js/components/Vis.tsx#L106
Replace `⏎····analyser,⏎····canvas,⏎····mode,⏎····colors,⏎··` with `analyser,·canvas,·mode,·colors`
|
|
ci:
packages/webamp/js/components/Vis.tsx#L106
React Hook useMemo has missing dependencies: 'doubled', 'isMWOpen', 'pixelDensity', 'renderHeight', and 'smallVis'. Either include them or remove the dependency array
|
|
ci
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/setup-node@v4, pnpm/action-setup@v2. Actions will be forced to run with Node.js 24 by default starting June 16th, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L134
'fft' is already declared in the upper scope on line 96 column 9
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L133
'analyser' is already declared in the upper scope on line 20 column 19
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L63
Do not nest ternary expressions
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L58
Do not nest ternary expressions
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L56
Do not nest ternary expressions
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L54
Do not nest ternary expressions
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L40
'sadata' is assigned a value but never used. Allowed unused vars must match /^_/u
|
|
ci:
packages/webamp/js/components/VisualizerEngine.ts#L20
'oscStyle' is assigned a value but never used. Allowed unused vars must match /^_/u
|
|
ci:
packages/webamp/js/components/Vis.tsx#L85
Do not nest ternary expressions
|
|
ci:
packages/webamp/js/components/Vis.tsx#L14
'preRenderBg' is defined but never used. Allowed unused vars must match /^_/u
|