Add eslint rules for consistent naming #64
thornbillpull_request.yml
on: pull_request_target
Create comments ✍️
/
CloudFlare Pages deployment 📃🚀
2s
Build 🏗️
/
Run production build 🏗️
3m 25s
Automation 🎛️
/
Merge conflict labeling 🏷️
3s
Quality checks 👌🧪
/
Vulnerable dependencies 🔎
6s
Run eslint suggestions
1m 26s
Matrix: GitHub CodeQL 🔬 / analyze
Matrix: Quality checks 👌🧪 / quality
Deploy 🚀
/
...
/
CloudFlare Pages deployment 📃🚀
3s
Annotations
10 errors, 10 warnings, and 4 notices
|
Quality checks 👌🧪 / Run lint 🕵️♂️:
src/apps/dashboard/routes/users/add.tsx#L15
Type Alias name `userInput` must match one of the following formats: PascalCase
|
|
Quality checks 👌🧪 / Run lint 🕵️♂️:
src/controllers/playback/video/index.js#L1944
Variable name `primary_icon_name` must match one of the following formats: camelCase, PascalCase, UPPER_CASE
|
|
Quality checks 👌🧪 / Run lint 🕵️♂️:
src/controllers/playback/video/index.js#L1945
Variable name `secondary_icon_name` must match one of the following formats: camelCase, PascalCase, UPPER_CASE
|
|
Quality checks 👌🧪 / Run lint 🕵️♂️:
src/controllers/playback/video/index.js#L1946
Variable name `animation_class` must match one of the following formats: camelCase, PascalCase, UPPER_CASE
|
|
Quality checks 👌🧪 / Run lint 🕵️♂️:
src/lib/legacy/domParserTextHtml.js#L15
Variable name `DOMParser_proto` must match one of the following formats: camelCase, PascalCase, UPPER_CASE
|
|
Quality checks 👌🧪 / Run lint 🕵️♂️:
src/lib/legacy/domParserTextHtml.js#L16
Variable name `real_parseFromString` must match one of the following formats: camelCase, PascalCase, UPPER_CASE
|
|
Quality checks 👌🧪 / Run lint 🕵️♂️:
src/lib/legacy/htmlMediaElement.js#L10
Variable name `HTMLMediaElement_proto` must match one of the following formats: camelCase, PascalCase, UPPER_CASE
|
|
Quality checks 👌🧪 / Run lint 🕵️♂️:
src/lib/legacy/htmlMediaElement.js#L11
Variable name `real_play` must match one of the following formats: camelCase, PascalCase, UPPER_CASE
|
|
Quality checks 👌🧪 / Run lint 🕵️♂️:
src/scripts/browser.js#L207
Variable name `platform_match` must match one of the following formats: camelCase, PascalCase, UPPER_CASE
|
|
Quality checks 👌🧪 / Run lint 🕵️♂️:
src/scripts/editorsidebar.js#L18
Object Literal Property name `li_attr` must match one of the following formats: camelCase, PascalCase
|
|
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/parentalcontrol.tsx#L225
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#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...'
|
|
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.6 MB |
|