Merge pull request #657 from 3flex/no-unnecessary-type-assertion

Enable no-unnecessary-type-assertion ts eslint rule
jellyfin · Nov 3, 2024 · 34937c9 · 34937c9
2 parents 6ac3432 + 9f9c456
commit 34937c9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -59,6 +59,7 @@ export default [
         rules: {
             '@typescript-eslint/explicit-function-return-type': 'error',
             '@typescript-eslint/no-explicit-any': 'warn',
+            '@typescript-eslint/no-unnecessary-type-assertion': 'error',
             '@typescript-eslint/no-unused-expressions': 'warn',
             '@typescript-eslint/no-unused-vars': 'error',
             '@typescript-eslint/prefer-ts-expect-error': 'error',

diff --git a/src/components/documentManager.ts b/src/components/documentManager.ts
@@ -581,9 +581,9 @@ export abstract class DocumentManager {
         const element = this.querySelector('.detailImageProgressContainer');
 
         if (value) {
-            (element as HTMLElement).classList.remove('d-none');
+            element.classList.remove('d-none');
         } else {
-            (element as HTMLElement).classList.add('d-none');
+            element.classList.add('d-none');
         }
     }