chore: Remove unused variables and enable no-unused-vars rule(#18584)

Merged
Merging
fix/no-unused-vars
into
main
-54%
IMPROVEMENTS
0
REGRESSIONS
1
UNTOUCHED
2
NEW
0
DROPPED
0

Benchmarks

Failing

typescript compilation ~50 Models
Regression
packages/client/src/__tests__/benchmarks/huge-schema/huge-schema.bench.ts::typescript::typescript compilation ~50 Models
-54%
63.8 ms
137.8 ms

Passing

client generation ~50 Models
packages/client/src/__tests__/benchmarks/huge-schema/huge-schema.bench.ts::typescript::client generation ~50 Models
+34%
7.9 s
5.9 s
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
0%
62.4 s
62.5 s

Commits

Click on a commit to change the comparison range
chore(deps): update definitelytyped (#18147) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
base
d117706
8 months ago by renovate[bot]
+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
8 months ago by SevInf
-7%
Update snapshots
97ec810
8 months ago by SevInf
-118%
Update more snapshots
c7dc5be
8 months ago by SevInf
+63%
Add comments
85ecd9a
8 months ago by SevInf
-28%
Add comments
a0555b8
8 months ago by SevInf
ResourcesHomePricingDocsBlogGitHub
Copyright © 2023 CodSpeed Technology SAS. All rights reserved.