refactor!: move Router fully to TypeScript

.eslintrc.json

@@ -1,77 +1,23 @@
 {
-  "extends": "eslint:recommended",
-  "env": {
-    "browser": true,
-    "node": true,
-    "es6": true
-  },
-  "plugins": [
-    "html"
+  "extends": [
+    "vaadin/typescript-requiring-type-checking",
+    "vaadin/imports-typescript",
+    "vaadin/prettier",
+    "vaadin/testing",
+    "plugin:oxlint/recommended"
   ],
-  "globals": {
-    "Polymer": false,
-    "Vaadin": false
-  },
   "parserOptions": {
-    "sourceType": "module"
+    "project": "./tsconfig.json"
   },
+  "plugins": ["tsdoc"],
   "rules": {
-    "curly": 2,
-    "no-caller": 2,
-    "no-extend-native": 2,
-    "no-extra-bind": 2,
-    "no-invalid-this": 2,
-    "no-multi-spaces": 2,
-    "no-new-wrappers": 2,
-    "no-with": 2,
-    "array-bracket-spacing": [2, "never"],
-    "brace-style": 2,
-
-    "camelcase": [2, {"properties": "never"}],
-    "comma-spacing": 2,
-    "comma-style": 2,
-    "computed-property-spacing": 2,
-    "func-call-spacing": 2,
-    "indent": [2, 2, {"SwitchCase": 1}],
-    "key-spacing": 2,
-    "keyword-spacing": 2,
-    "linebreak-style": 2,
-    "max-len": [2, {"code": 140}],
-    "no-array-constructor": 2,
-    "no-multiple-empty-lines": [2, {"max": 2}],
-    "no-new-object": 2,
-    "no-trailing-spaces": 2,
-    "object-curly-spacing": 2,
-    "quote-props": [2, "consistent"],
-    "quotes": [2, "single", {"allowTemplateLiterals": true}],
-    "semi-spacing": 2,
-    "semi": 2,
-    "space-before-blocks": 2,
-    "space-before-function-paren": [2, "never"],
-    "space-in-parens": 2,
-    "space-infix-ops": 2,
-    "spaced-comment": [2, "always"],
-
-    "arrow-spacing": 2,
-    "constructor-super": 2,
-    "no-confusing-arrow": 2,
-    "no-const-assign": 2,
-    "no-dupe-class-members": 2,
-    "no-duplicate-imports": 2,
-    "no-this-before-super": 2,
-    "no-useless-computed-key": 2,
-    "no-useless-constructor": 2,
-    "no-useless-rename": 2,
-    "prefer-const": 2,
-    "rest-spread-spacing": 2,
-    "template-curly-spacing": 2,
-
-    "no-unused-vars": ["error", { "args": "none" }]
+    "@typescript-eslint/no-invalid-void-type": "off",
+    "@typescript-eslint/no-useless-template-literals": "off",
+    "import/no-unassigned-import": "off",
+    "max-params": "off",
+    "sort-keys": "off",
+    "tsdoc/syntax": "error",
+    "import/prefer-default-export": "off"
   },
-  "overrides": [{
-    "files": ["gulpfile.js"],
-    "parserOptions": {
-      "ecmaVersion": 2017
-    }
-  }]
+  "ignorePatterns": ["*.cjs"]
 }

.github/workflows/validation.yml

@@ -0,0 +1,34 @@
+name: Validation
+
+on:
+  push:
+    branches:
+      - 'master'
+  pull_request:
+permissions:
+  contents: read
+
+jobs:
+  test:
+    name: Validation
+    runs-on: ubuntu-latest
+    timeout-minutes: 15
+
+    steps:
+      - name: Checkout Project Code
+        uses: actions/checkout@v3
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
+          fetch-depth: 0
+      - name: Setup Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: 'lts/*'
+      - name: Install dependencies and build
+        run: npm ci
+      - name: Build TypeScript
+        run: npm run build
+      - name: Check TypeScript
+        run: npm run typecheck
+      - name: Test
+        run: npm test

.gitignore

@@ -1,7 +1,6 @@
 bower_components
 node_modules
 build
-package-lock.json
 dist
 coverage
 

.prettierrc.json

@@ -0,0 +1,7 @@
+{
+  "bracketSpacing": true,
+  "printWidth": 120,
+  "trailingComma": "all",
+  "tabWidth": 2,
+  "singleQuote": true
+}

.run/All tests.run.xml

@@ -0,0 +1,10 @@
+<component name="ProjectRunConfigurationManager">
+  <configuration default="false" name="All tests" type="JavaScriptTestRunnerKarma">
+    <config-file value="$PROJECT_DIR$/karma.config.cjs" />
+    <karma-package-dir value="$PROJECT_DIR$/node_modules/karma" />
+    <working-directory value="$PROJECT_DIR$" />
+    <node-interpreter value="project" />
+    <envs />
+    <method v="2" />
+  </configuration>
+</component>

.run/Template Karma.run.xml

@@ -0,0 +1,10 @@
+<component name="ProjectRunConfigurationManager">
+  <configuration default="true" type="JavaScriptTestRunnerKarma">
+    <config-file value="$PROJECT_DIR$/karma.config.cjs" />
+    <karma-package-dir value="$PROJECT_DIR$/node_modules/karma" />
+    <working-directory value="$PROJECT_DIR$" />
+    <node-interpreter value="project" />
+    <envs />
+    <method v="2" />
+  </configuration>
+</component>

README.md

@@ -66,7 +66,7 @@ Cross-browser Testing Platform and Open Source <3 Provided by [Sauce Labs](https
 
   - [http://127.0.0.1:8000/components/vaadin-router/demo](http://127.0.0.1:8000/components/vaadin-router/demo)
   - Public API tests: [http://127.0.0.1:8000/components/vaadin-router/test](http://127.0.0.1:8000/components/vaadin-router/test)
-  - Unit tests: [http://127.0.0.1:8000/components/vaadin-router/test/index.coverage.html](http://127.0.0.1:8000/components/vaadin-router/test/index.coverage.html)
+  - Unit tests: [http://127.0.0.1:8000/components/vaadin-router/test/index.html](http://127.0.0.1:8000/components/vaadin-router/test/index.html)
 
 
 ## Running tests from the command line

demo/demo-shell.html

@@ -8,10 +8,10 @@
 <link rel="import" href="element-demo.html">
 <link rel="import" href="vaadin-router-demo-shared-styles.html">
 
-<script src="../dist/vaadin-router.umd.js"></script>
-<script>
-  // In order to make the demos work without the import statement there
-  window.Router = window.Vaadin.Router;
+<script type="module">
+  import {Router} from '../dist/vaadin-router.js';
+
+  window.Router = Router;
 </script>
 
 <link rel="import" href="demo-elements/x-home-view.html">