Comparing
fix/no-unused-vars
(a0555b8
) withmain
(1a0fbfa
)-52%
- Improvements
- 1
- Regressions
- 1
- Untouched
- 1
- New
- 0
- Dropped
- 0
- Ignored
- 0
Benchmarks
Failed
typescript compilation ~50 ModelsRegression
packages/client/src/__tests__/benchmarks/huge-schema/huge-schema.bench.ts::typescript::typescript compilation ~50 Models-52%
14.9 ms
31 ms
Improved
client generation ~50 Models
packages/client/src/__tests__/benchmarks/huge-schema/huge-schema.bench.ts::typescript::client generation ~50 Models+34%
1.6 s
1.2 s
Passed
client generation 100 models with relations
packages/client/src/__tests__/benchmarks/lots-of-relations/lots-of-relations.bench.ts::typescript::client generation 100 models with relations+1%
11.9 s
11.8 s
Commits
Click on a commit to change the comparison rangeBase
main
1a0fbfa
+36%
chore: Remove unused variables and enable no-unused-vars rule
Enables `@typesctipt-eslint/no-unused-vars` and fixes all violations.
There are a couple of exceptions to the rule:
1. Spreads are ignored, making it possible to omit a property with
object spread operator, i.e. this code valid:
```ts
const { propToOmit, ...rest } = someObject
```
2. In some cases, we want to have a function argument that won't be used
in the function body. Main use-case for this is TS inference tests.
In that case, argument may be prefixed with `_` and the rule won't
trigger.
82320ac
2 years ago
by SevInf+4%
Update snapshots
97ec810
2 years ago
by SevInf-128%
Update more snapshots
c7dc5be
2 years ago
by SevInf+64%
Add comments
85ecd9a
2 years ago
by SevInf-29%
Add comments
a0555b8
2 years ago
by SevInf