chore: update dependencies & lint rules (#142)

Author: json-derulo

.eslintrc.json

@@ -9,6 +9,8 @@
         "createDefaultProgram": true
       },
       "extends": [
+        "plugin:@typescript-eslint/strict-type-checked",
+        "plugin:@typescript-eslint/stylistic-type-checked",
         "plugin:@angular-eslint/recommended",
         "plugin:@angular-eslint/template/process-inline-templates"
       ],
@@ -31,6 +33,12 @@
         ]
       }
     },
+    {
+      "files": ["*.spec.ts"],
+      "rules": {
+        "@typescript-eslint/dot-notation": "off"
+      }
+    },
     {
       "files": ["*.html"],
       "extends": ["plugin:@angular-eslint/template/recommended"],

package-lock.json

@@ -28,7 +28,8 @@
     "major-release": "npm run major-release_",
     "dryBuild": "run-s lib demo",
     "update": "npx npm-check-updates -u",
-    "prettier-check": "prettier . --check"
+    "prettier-check": "prettier . --check",
+    "prettier-fix": "prettier . --fix"
   },
   "dependencies": {
     "@angular/animations": "^16.0.0",
@@ -58,12 +59,12 @@
     "@angular/compiler-cli": "^16.0.0",
     "@angular/language-service": "^16.0.0",
     "@types/jasmine": "^4.3.0",
-    "@typescript-eslint/eslint-plugin": "^5.59.2",
-    "@typescript-eslint/parser": "^5.59.2",
+    "@typescript-eslint/eslint-plugin": "^6.1.0",
+    "@typescript-eslint/parser": "^6.1.0",
     "copyfiles": "^2.1.1",
     "eslint": "^8.39.0",
     "gh-pages": "^5.0.0",
-    "jasmine-core": "^4.5.0",
+    "jasmine-core": "^5.0.1",
     "karma": "^6.4.1",
     "karma-chrome-launcher": "~3.2.0",
     "karma-coverage": "^2.2.0",
@@ -72,6 +73,6 @@
     "ng-packagr": "^16.0.0",
     "npm-run-all": "^4.1.5",
     "prettier": "^3.0.0",
-    "typescript": "~5.0.4"
+    "typescript": "~5.1.6"
   }
 }

package.json

@@ -24,7 +24,7 @@ describe("MaterialCssVarsService", () => {
       ],
     });
     service = TestBed.inject(MaterialCssVarsService);
-    renderer = (service as any).renderer;
+    renderer = service["renderer"];
   });
 
   it("should be created", () => {

projects/material-css-vars/src/lib/material-css-vars.service.spec.ts

@@ -163,6 +163,8 @@ export class MaterialCssVarsService {
     threshold: HueValue,
     palettePrefix: MatCssPalettePrefix,
   ) {
+    // ToDo: address this lint error
+    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
     if (!threshold || !palettePrefix || this.isAutoContrast) {
       return;
     }
@@ -345,7 +347,7 @@ export class MaterialCssVarsService {
     const isContrast: boolean = name.includes(
       MaterialCssVarsService.CONTRAST_PREFIX,
     );
-    let noRgb: string = "";
+    let noRgb = "";
     if (isContrast) {
       noRgb = value.replace(")", "-no-rgb)");
     } else {

projects/material-css-vars/src/lib/material-css-vars.service.ts

@@ -17,9 +17,8 @@ class ButtonComponent {
 function getButtonComputedStyle(
   fixture: ComponentFixture<ButtonComponent>,
 ): CSSStyleDeclaration {
-  const buttonElement = fixture.debugElement.query(
-    By.css("button"),
-  ).nativeElement;
+  const buttonElement = fixture.debugElement.query(By.css("button"))
+    .nativeElement as HTMLButtonElement;
   return getComputedStyle(buttonElement);
 }
 

projects/material-css-vars/src/test/integration.spec.ts

@@ -45,7 +45,7 @@ export class AppComponent {
   ];
 
   spinnerMode: ProgressSpinnerMode = "indeterminate";
-  spinnerValue: number = 25;
+  spinnerValue = 25;
   spinnerColor: ThemePalette = "primary";
   availableSpinnerColors = [
     { name: "none", color: "" },
@@ -67,7 +67,7 @@ export class AppComponent {
     ],
   };
 
-  allComplete: boolean = false;
+  allComplete = false;
 
   constructor(
     private _dialog: MatDialog,

src/app/app.component.ts

@@ -85,7 +85,8 @@ import { MaterialCssVarsModule } from "../../projects/material-css-vars/src/lib/
       warn: "#f44336",
     }),
   ],
-  providers: [{ provide: APP_BASE_HREF, useValue: window["_app_base"] || "/" }],
+  providers: [{ provide: APP_BASE_HREF, useValue: window._app_base ?? "/" }],
   bootstrap: [AppComponent],
 })
+// eslint-disable-next-line @typescript-eslint/no-extraneous-class
 export class AppModule {}

src/app/app.module.ts

@@ -10,7 +10,9 @@ if (environment.production) {
 
 platformBrowserDynamic()
   .bootstrapModule(AppModule)
-  .catch((err) => console.error(err));
+  .catch((err) => {
+    console.error(err);
+  });
 
 declare global {
   interface Window {