Skip to content

Migrate resume & streaming to React #868

Migrate resume & streaming to React

Migrate resume & streaming to React #868

Triggered via pull request February 17, 2025 18:27
@viownviown
synchronize #6516
Status Success
Total duration 4m 20s
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 27s
Build 🏗️ / Run production build 🏗️
Automation 🎛️  /  Merge conflict labeling 🏷️
5s
Automation 🎛️ / Merge conflict labeling 🏷️
Quality checks 👌🧪  /  Vulnerable dependencies 🔎
7s
Quality checks 👌🧪 / Vulnerable dependencies 🔎
Run eslint suggestions
1m 13s
Run eslint suggestions
Matrix: GitHub CodeQL 🔬 / analyze
Matrix: Quality checks 👌🧪 / quality
Deploy 🚀  /  ...  /  CloudFlare Pages deployment 📃🚀
3s
Deploy 🚀 / Compose and push comment 📝 / CloudFlare Pages deployment 📃🚀
Fit to window
Zoom out
Zoom in

Annotations

11 warnings and 4 notices
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/controllers/dashboard.js#L485
Unexpected 'fixme' comment: 'FIXME: It seems that, sometimes, server...'
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#L130
React Hook useCallback has a missing dependency: 'setTitle'. Either include it or remove the dependency array
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
GitHub CodeQL 🔬 / Analyze javascript-typescript 🔬
Failed to save: Failed to CreateCacheEntry: Received non-retryable error: Failed request: (409) Conflict: cache entry with the same key, version, and scope already exists
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
Run eslint suggestions
ESLint passes

Artifacts

Produced during runtime
Name Size
frontend
34 MB