Skip to content

Commit de94712

Browse files
committed
chore: use biome for linting
Signed-off-by: ferhat elmas <elmas.ferhat@gmail.com>
1 parent 91fefd6 commit de94712

File tree

22 files changed

+68
-1554
lines changed

22 files changed

+68
-1554
lines changed

.eslintrc.js

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

.github/workflows/ci.yml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,14 +43,10 @@ jobs:
4343
run: |
4444
npm ci
4545
46-
- name: Biome format checks
46+
- name: Lint
4747
run: |
4848
npm run lint
4949
50-
- name: ESLint checks
51-
run: |
52-
npm run eslint:check
53-
5450
- name: Builds successfully
5551
run: |
5652
npm run build

README.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,5 @@ To perform your tests you can run the following command: `npm test`
6969

7070
### Code Quality
7171

72-
- Apply formatting: `npm run format`
73-
- Check formatting: `npm run lint`
74-
- Run ESLint rules: `npm run eslint:check`
72+
- Check: `npm run lint`
73+
- Automatic fix if possible: `npm run lint:fix`

biome.jsonc

Lines changed: 31 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,21 @@
11
{
22
"$schema": "https://biomejs.dev/schemas/2.4.4/schema.json",
33
"files": {
4-
"includes": ["src/**/*.ts", "*.js", "*.cjs", "*.mjs", "*.json", "*.jsonc", "!package-lock.json"]
4+
"includes": [
5+
"**/*.ts",
6+
"**/*.js",
7+
"**/*.cjs",
8+
"**/*.mjs",
9+
"**/*.json",
10+
"**/*.jsonc",
11+
"!package-lock.json",
12+
"!build/**",
13+
"!coverage/**",
14+
"!dist/**",
15+
"!static/**",
16+
"!data/**",
17+
"!node_modules/**"
18+
]
519
},
620
"formatter": {
721
"enabled": true,
@@ -10,7 +24,22 @@
1024
"lineWidth": 100
1125
},
1226
"linter": {
13-
"enabled": false
27+
"enabled": true,
28+
"rules": {
29+
"recommended": false,
30+
"correctness": {
31+
"noUnusedVariables": "warn"
32+
},
33+
"style": {
34+
"noCommonJs": "warn"
35+
},
36+
"suspicious": {
37+
"noExplicitAny": "warn"
38+
},
39+
"nursery": {
40+
"noFloatingPromises": "error"
41+
}
42+
}
1443
},
1544
"assist": {
1645
"enabled": false

build.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// eslint-disable-next-line @typescript-eslint/no-var-requires
1+
// biome-ignore lint/style/noCommonJs: build script runs as CommonJS
22
const { build } = require('esbuild')
33

44
build({

jest.sequencer.cjs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
// eslint-disable-next-line @typescript-eslint/no-var-requires
21
const Sequencer = require('@jest/test-sequencer').default
32

43
// order to sort tests based on name matching

0 commit comments

Comments
 (0)