Skip to content

Separate and refactor SubtitleSync and OffsetController #2109

Separate and refactor SubtitleSync and OffsetController

Separate and refactor SubtitleSync and OffsetController #2109

Triggered via pull request July 20, 2025 18:51
@armanckeserarmanckeser
synchronize #6824
Status Success
Total duration 4m 13s
Artifacts 1

pull_request.yml

on: pull_request_target
Create comments ✍️  /  CloudFlare Pages deployment 📃🚀
7s
Create comments ✍️ / CloudFlare Pages deployment 📃🚀
Build 🏗️  /  Run production build 🏗️
3m 25s
Build 🏗️ / Run production build 🏗️
Automation 🎛️  /  Merge conflict labeling 🏷️
5s
Automation 🎛️ / Merge conflict labeling 🏷️
Quality checks 👌🧪  /  Vulnerable dependencies 🔎
8s
Quality checks 👌🧪 / Vulnerable dependencies 🔎
Run eslint suggestions
3m 3s
Run eslint suggestions
Matrix: GitHub CodeQL 🔬 / analyze
Matrix: Quality checks 👌🧪 / quality
Deploy 🚀  /  ...  /  CloudFlare Pages deployment 📃🚀
7s
Deploy 🚀 / Compose and push comment 📝 / CloudFlare Pages deployment 📃🚀
Fit to window
Zoom out
Zoom in

Annotations

10 warnings and 4 notices
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/routes/users/add.tsx#L126
Refactor this code to not nest functions more than 4 levels deep
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/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/branding/index.tsx#L127
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/features/sessions/utils/getNowPlayingName.ts#L17
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/features/logs/api/useServerLog.ts#L14
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/features/backups/api/useRestoreBackup.ts#L7
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/features/backups/api/useCreateBackup.ts#L10
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/features/backups/api/useBackups.ts#L10
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/components/AppTabs.tsx#L56
Unexpected 'hack' comment: 'HACK: Force resizing to workaround...'
Run eslint suggestions
ESLint passes
Run eslint suggestions
Using ESLint with default configuration
Run eslint suggestions
ESLint version: 9.30.1
Run eslint suggestions
Using ESLint from: /home/runner/work/jellyfin-web/jellyfin-web/node_modules/eslint/lib/api.js

Artifacts

Produced during runtime
Name Size Digest
frontend
33.8 MB
sha256:c7c683e8ccc6d5d055398e4adcf447eed7108a4dda1298a02ba0ec1121bb80f8