Skip to content
Pull Request 📥

Smart font streaming for external ASS subtitles #177

Sign in to view logs

Smart font streaming for external ASS subtitles

Smart font streaming for external ASS subtitles #177

Triggered via pull request October 25, 2024 22:59
@p0358p0358
opened #6257
Status Failure
Total duration 4m 17s
Artifacts 1

pull_request.yml

on: pull_request_target
Create comments ✍️  /  CloudFlare Pages deployment 📃🚀
2s
Create comments ✍️ / CloudFlare Pages deployment 📃🚀
Build 🏗️  /  Run production build 🏗️
3m 22s
Build 🏗️ / Run production build 🏗️
Automation 🎛️  /  Merge conflict labeling 🏷️
3s
Automation 🎛️ / Merge conflict labeling 🏷️
Quality checks 👌🧪  /  Vulnerable dependencies 🔎
5s
Quality checks 👌🧪 / Vulnerable dependencies 🔎
Run eslint suggestions
1m 27s
Run eslint suggestions
Matrix: GitHub CodeQL 🔬 / analyze
Matrix: Quality checks 👌🧪 / quality
Deploy 🚀  /  CloudFlare Pages 📃
28s
Deploy 🚀 / CloudFlare Pages 📃
https://2d731c5a.jellyfin-web.pages.dev
Deploy 🚀  /  ...  /  CloudFlare Pages deployment 📃🚀
3s
Deploy 🚀 / Compose and push comment 📝 / CloudFlare Pages deployment 📃🚀
Fit to window
Zoom out
Zoom in

Annotations

11 errors, 10 warnings, and 3 notices
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/plugins/htmlVideoPlayer/plugin.js#L169
Expected { after 'if' condition
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/plugins/htmlVideoPlayer/plugin.js#L172
Expected { after 'if' condition
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/plugins/htmlVideoPlayer/plugin.js#L174
'serverId' is not defined
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/plugins/htmlVideoPlayer/plugin.js#L176
Unnecessary escape character: \-
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/plugins/htmlVideoPlayer/plugin.js#L176
Unnecessary escape character: \?
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/plugins/htmlVideoPlayer/plugin.js#L177
'textStream' is not defined
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/plugins/htmlVideoPlayer/plugin.js#L178
Expected { after 'if' condition
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/plugins/htmlVideoPlayer/plugin.js#L180
'textStream' is not defined
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/plugins/htmlVideoPlayer/plugin.js#L1305
'encodingOptions' is not defined
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/plugins/htmlVideoPlayer/plugin.js#L1308
'encodingOptions' is defined but never used
Run eslint suggestions
ESLint fails
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/components/AppTabs.tsx#L56
Unexpected 'hack' comment: 'HACK: Force resizing to workaround...'
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/routes/users/access.tsx#L141
React Hook useCallback has a missing dependency: 'libraryMenu'. Either include it or remove the dependency array
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/routes/users/access.tsx#L239
React Hook useEffect has a missing dependency: 'userId'. Either include it or remove the dependency array
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/routes/users/profile.tsx#L176
React Hook useCallback has a missing dependency: 'libraryMenu'. Either include it or remove the dependency array
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/routes/users/profile.tsx#L185
React Hook useCallback has a missing dependency: 'getUser'. Either include it or remove the dependency array
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/routes/users/profile.tsx#L264
React Hook useEffect has a missing dependency: 'getUser'. Either include it or remove the dependency array
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/experimental/components/tabs/AppTabs.tsx#L25
Unexpected 'hack' comment: 'HACK: Force resizing to workaround...'
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/experimental/routes/home.tsx#L126
React Hook useCallback has a missing dependency: 'setTitle'. Either include it or remove the dependency array
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/stable/features/playback/utils/mediaSegmentManager.ts#L28
Unexpected 'fixme' comment: 'FIXME: Replace with SDK...'
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/stable/routes/user/userprofile.tsx#L75
React Hook useCallback has a missing dependency: 'libraryMenu'. Either include it or remove the dependency array
Run eslint suggestions
Using ESLint from: /home/runner/work/jellyfin-web/jellyfin-web/node_modules/eslint/lib/api.js
Run eslint suggestions
ESLint version: 8.57.1
Run eslint suggestions
Using ESLint with default configuration

Artifacts

Produced during runtime
Name Size
frontend
34.6 MB