Skip to content

Commit 7b77258

Browse files
authored
Merge pull request #87 from tiagosiebler/linter
chore(v1.5.0): bump build version to node LTS, fix linter setup, fix webpack setup, address linter conflicts
2 parents aadb034 + 7cc0fb9 commit 7b77258

32 files changed

+2003
-973
lines changed

.eslintrc.cjs

Lines changed: 53 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,53 @@
1+
module.exports = {
2+
parser: '@typescript-eslint/parser',
3+
parserOptions: {
4+
project: 'tsconfig.linting.json',
5+
tsconfigRootDir: __dirname,
6+
sourceType: 'module',
7+
},
8+
plugins: [
9+
'@typescript-eslint/eslint-plugin',
10+
'simple-import-sort',
11+
// 'require-extensions', // only once moved to ESM
12+
],
13+
extends: [
14+
'plugin:@typescript-eslint/recommended',
15+
'plugin:prettier/recommended',
16+
// 'plugin:require-extensions/recommended', // only once moved to ESM
17+
],
18+
root: true,
19+
env: {
20+
node: true,
21+
jest: true,
22+
},
23+
ignorePatterns: ['.eslintrc.js', 'webpack.config.js'],
24+
rules: {
25+
'@typescript-eslint/interface-name-prefix': 'off',
26+
'@typescript-eslint/explicit-function-return-type': 'off',
27+
'@typescript-eslint/explicit-module-boundary-types': 'off',
28+
'@typescript-eslint/no-explicit-any': 'off',
29+
'@typescript-eslint/no-non-null-assertion': 'off',
30+
'@typescript-eslint/ban-types': 'off',
31+
'simple-import-sort/imports': 'error',
32+
'simple-import-sort/exports': 'error',
33+
'array-bracket-spacing': ['error', 'never'],
34+
'linebreak-style': ['error', 'unix'],
35+
'lines-between-class-members': ['warn', 'always'],
36+
semi: ['error', 'always'],
37+
'new-cap': 'off',
38+
'no-console': 'off',
39+
'no-debugger': 'off',
40+
'no-mixed-spaces-and-tabs': 2,
41+
'no-use-before-define': [2, 'nofunc'],
42+
'no-unreachable': ['warn'],
43+
// 'no-unused-vars': ['warn'],
44+
'no-extra-parens': ['off'],
45+
'no-mixed-operators': ['off'],
46+
quotes: [2, 'single', 'avoid-escape'],
47+
'block-scoped-var': 2,
48+
'brace-style': [2, '1tbs', { allowSingleLine: true }],
49+
'computed-property-spacing': [2, 'never'],
50+
'keyword-spacing': 2,
51+
'space-unary-ops': 2,
52+
},
53+
};

.eslintrc.js

Lines changed: 0 additions & 36 deletions
This file was deleted.

.github/workflows/e2etest.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,12 @@ jobs:
3030
- name: Install
3131
run: npm ci --ignore-scripts
3232

33-
- name: Build
33+
- name: Check Build
3434
run: npm run build
3535

36+
- name: Check Lint
37+
run: npm run lint
38+
3639
- name: Test
3740
run: npm run test
3841
env:

.nvmrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
v20.11.0
1+
v22.11.0

0 commit comments

Comments
 (0)