Merge pull request #66 from github/upgrade-to-new-web-component-standards

upgrade to new web component standards

Author: keithamus

.eslintrc.json

@@ -1,22 +1,40 @@
 {
-  "parser": "@typescript-eslint/parser",
+  "root": true,
   "extends": [
     "plugin:github/browser",
     "plugin:github/recommended",
-    "plugin:github/typescript"
+    "plugin:github/typescript",
+    "plugin:custom-elements/recommended"
   ],
-  "globals": {
-    "TabContainerElement": "readable"
-  },
   "rules": {
-    "@typescript-eslint/no-non-null-assertion": "off"
+    "custom-elements/tag-name-matches-class": [
+      "error",
+      {
+        "suffix": "Element"
+      }
+    ],
+    "custom-elements/define-tag-after-class-definition": "off",
+    "custom-elements/no-method-prefixed-with-on": "off",
+    "custom-elements/expose-class-on-global": "off",
+    "import/extensions": ["error", "always"],
+    "import/no-unresolved": "off"
   },
   "overrides": [
     {
-      "files": ["test/**/*.js", "prettier.config.js"],
-      "parser": "espree",
+      "files": "src/*-define.ts",
+      "rules": {
+        "@typescript-eslint/no-namespace": "off"
+      }
+    },
+    {
+      "files": "test/**/*.js",
       "rules": {
-        "github/unescaped-html-literal": "off"
+        "github/unescaped-html-literal": "off",
+        "github/no-inner-html": "off",
+        "i18n-text/no-en": "off"
+      },
+      "env": {
+        "mocha": true
       }
     }
   ]

.github/workflows/nodejs.yml

@@ -1,46 +1,19 @@
-name: Build
-
-on:
-  pull_request:
-  push:
-    branches:
-      - main
+name: Node CI
 
+on: push
 jobs:
   build:
-    name: Build
     runs-on: ubuntu-latest
     steps:
-      - name: Get sources
-        uses: actions/checkout@v1
-
-      - name: Restore npm cache
-        uses: actions/cache@v2
+      - uses: actions/checkout@v2
+      - name: Use Node.js 18.x
+        uses: actions/setup-node@v1
         with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-
-
-      - uses: actions/cache@v2
-        with:
-          path: node_modules
-          key: ${{ runner.os }}-node-modules-${{ hashFiles('**/package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-modules-
-
-      - name: Use Node.js 14
-        uses: actions/setup-node@v2
-        with:
-          node-version: '14'
-
-      - name: Install dependencies
-        run: npm install
-
-      - name: Build
-        run: npm run build
-
-      - name: Test
-        run: npm run test
+          node-version: 18.x
+      - name: npm install, build, and test
+        run: |
+          npm install
+          npm run build --if-present
+          npm test
         env:
           CI: true

.github/workflows/publish.yml

@@ -11,7 +11,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: 14
+          node-version: 18
           registry-url: https://registry.npmjs.org/
           cache: npm
       - run: npm ci