Merge pull request #588 from GetStream/v5-update-angular

Drop support for Angular 12-14

Author: szuperaz

.eslintrc.json

@@ -1,10 +1,7 @@
 {
   "root": true,
   "ignorePatterns": ["projects/**/*"],
-  "plugins": ["unused-imports", "cypress", "jasmine"],
-  "env": {
-    "jasmine": true
-  },
+  "plugins": ["unused-imports"],
   "overrides": [
     {
       "files": ["*.ts"],
@@ -16,19 +13,38 @@
         "eslint:recommended",
         "plugin:@typescript-eslint/recommended-requiring-type-checking",
         "plugin:@angular-eslint/all",
-        "plugin:cypress/recommended",
-        "plugin:jasmine/recommended",
         "plugin:prettier/recommended" // This should always be the last in the extends array
       ],
       "rules": {
         "@angular-eslint/sort-ngmodule-metadata-arrays": "off",
         "@angular-eslint/prefer-on-push-component-change-detection": "off",
         "unused-imports/no-unused-imports": "error",
         "no-console": ["error", { "allow": ["warn", "error"] }],
-        "jasmine/no-spec-dupes": [1, "branch"],
         "@typescript-eslint/unbound-method": "off"
       }
     },
+    {
+      "files": ["*.spec.ts", "**/mocks/*.ts"],
+      "parserOptions": {
+        "project": ["tsconfig.json"],
+        "createDefaultProgram": true
+      },
+      "extends": [
+        "eslint:recommended",
+        "plugin:@typescript-eslint/recommended-requiring-type-checking",
+        "plugin:@angular-eslint/all",
+        "plugin:prettier/recommended" // This should always be the last in the extends array
+      ],
+      "rules": {
+        "@angular-eslint/sort-ngmodule-metadata-arrays": "off",
+        "@angular-eslint/prefer-on-push-component-change-detection": "off",
+        "unused-imports/no-unused-imports": "error",
+        "no-console": ["error", { "allow": ["warn", "error"] }],
+        "@typescript-eslint/unbound-method": "off",
+        "@typescript-eslint/no-explicit-any": "off",
+        "@typescript-eslint/ban-types": "off"
+      }
+    },
     {
       "files": ["*.html"],
       "extends": [
@@ -39,7 +55,11 @@
         "@angular-eslint/template/i18n": "off",
         "@angular-eslint/template/cyclomatic-complexity": "off",
         "@angular-eslint/template/no-call-expression": "off",
-        "@angular-eslint/template/no-autofocus": "off"
+        "@angular-eslint/template/no-autofocus": "off",
+        "@angular-eslint/template/no-interpolation-in-attributes": "off",
+        "@angular-eslint/template/button-has-type": "off",
+        "@angular-eslint/template/no-inline-styles": "off",
+        "@angular-eslint/template/accessibility-interactive-supports-focus": "off"
       }
     }
   ]

.github/workflows/workflow.yml

@@ -15,7 +15,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [16.x]
+        node-version: [18.x]
 
     steps:
       - uses: actions/checkout@v2
@@ -45,13 +45,6 @@ jobs:
       - name: Unit tests
         run: |
           npm run test:ci
-      # - name: E2e tests
-      #   env:
-      #     STREAM_API_KEY: ${{ secrets.API_KEY }}
-      #     STREAM_USER_ID: ${{ secrets.USER_ID }}
-      #     STREAM_USER_TOKEN: ${{ secrets.USER_TOKEN }}
-      #   run: |
-      #     npm run e2e:ci
       - name: Release
         env:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.gitignore

@@ -31,6 +31,7 @@ chrome-profiler-events*.json
 .history/*
 
 # misc
+/.angular/cache
 /.sass-cache
 /connect.lock
 /coverage

angular.json

@@ -1,7 +1,8 @@
 {
   "$schema": "./node_modules/@angular/cli/lib/config/schema.json",
   "cli": {
-    "defaultCollection": "@angular-eslint/schematics"
+    "schematicCollections": ["@angular-eslint/schematics"],
+    "analytics": false
   },
   "version": 1,
   "newProjectRoot": "projects",
@@ -323,5 +324,12 @@
       }
     }
   },
-  "defaultProject": "stream-chat-angular"
+  "schematics": {
+    "@angular-eslint/schematics:application": {
+      "setParserOptionsProject": true
+    },
+    "@angular-eslint/schematics:library": {
+      "setParserOptionsProject": true
+    }
+  }
 }

copy-generated-service-docs.ts

@@ -31,7 +31,10 @@ fs.readdir(sourcePath, (err: any, files: string[]) => {
       }
 
       // Remove the thre prefix from the title
-      const result = data.replace(/# Class:/g, '#').replace('<T\\>', '');
+      const result = data
+        .replace(/# Class:/g, '#')
+        .replace('<T\\>', '')
+        .replace('\\', '');
 
       fs.writeFile(`${sourcePath}/${file}`, result, 'utf8', (err: any) => {
         if (err) {

docusaurus/angular_versioned_docs/version-5/basics/upgrade-v4.mdx

@@ -0,0 +1,19 @@
+---
+id: upgrade-v4
+title: Upgrade from v4
+---
+
+## About version 5
+
+This version takes care of some important housekeeping tasks that help provide better maintainability on the long run.
+
+## Breaking changes
+
+### Angular version support
+
+Angular versions 12-14 are no longer supported. You can use the (Angular update guide)[https://update.angular.io/] to update your application.
+
+## Type changes
+
+- Event handlers with `Function` type are changed to `() => void`
+- Event handlers with `any` return types are changed to `void` return type