Skip to content

Commit f70862f

Browse files
authored
Merge pull request #67 from github/upgrade-to-new-web-component-standards
upgrade to new web component standards
2 parents 4e3b7ce + ed54c11 commit f70862f

14 files changed

+9450
-3935
lines changed

.eslintrc.json

Lines changed: 27 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,41 @@
11
{
2+
"root": true,
23
"extends": [
3-
"plugin:github/recommended",
44
"plugin:github/browser",
5-
"plugin:github/typescript"
5+
"plugin:github/recommended",
6+
"plugin:github/typescript",
7+
"plugin:custom-elements/recommended"
68
],
7-
"globals": {
8-
"ClipboardCopyElement": "readable"
9+
"rules": {
10+
"custom-elements/tag-name-matches-class": [
11+
"error",
12+
{
13+
"suffix": "Element"
14+
}
15+
],
16+
"custom-elements/define-tag-after-class-definition": "off",
17+
"custom-elements/no-method-prefixed-with-on": "off",
18+
"custom-elements/expose-class-on-global": "off",
19+
"import/extensions": ["error", "always"],
20+
"import/no-unresolved": "off"
921
},
1022
"overrides": [
23+
{
24+
"files": "src/*-define.ts",
25+
"rules": {
26+
"@typescript-eslint/no-namespace": "off"
27+
}
28+
},
1129
{
1230
"files": "test/**/*.js",
31+
"rules": {
32+
"github/unescaped-html-literal": "off",
33+
"github/no-inner-html": "off",
34+
"i18n-text/no-en": "off"
35+
},
1336
"env": {
1437
"mocha": true
15-
},
16-
"globals": {
17-
"assert": true
1838
}
19-
},
20-
{
21-
"files": "*.js",
22-
"parser": "espree",
23-
"parserOptions": {"ecmaVersion": 8}
2439
}
2540
]
2641
}

.github/workflows/nodejs.yml

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,15 @@ jobs:
55
build:
66
runs-on: ubuntu-latest
77
steps:
8-
- uses: actions/checkout@v2
9-
- name: Use Node.js 12.x
10-
uses: actions/setup-node@v1
11-
with:
12-
node-version: 12.x
13-
- name: npm install, build, and test
14-
run: |
15-
npm install
16-
npm run build --if-present
17-
npm test
18-
env:
19-
CI: true
8+
- uses: actions/checkout@v2
9+
- name: Use Node.js 18.x
10+
uses: actions/setup-node@v1
11+
with:
12+
node-version: 18.x
13+
- name: npm install, build, and test
14+
run: |
15+
npm install
16+
npm run build --if-present
17+
npm test
18+
env:
19+
CI: true

.github/workflows/publish.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ jobs:
1111
- uses: actions/checkout@v3
1212
- uses: actions/setup-node@v3
1313
with:
14-
node-version: 14
14+
node-version: 18
1515
registry-url: https://registry.npmjs.org/
1616
cache: npm
1717
- run: npm ci

0 commit comments

Comments
 (0)