fix eslint complaints

Author: hoeck

benchmarks/helpers/main.ts

@@ -28,7 +28,6 @@ export async function runAllBenchmarks() {
     return;
   }
 
-  const majorVersion = getNodeMajorVersion();
   const allResults: BenchmarkResult[] = [];
 
   for (const [benchmark, benchmarks] of getRegisteredBenchmarks()) {

index.ts

@@ -17,7 +17,7 @@ async function main() {
 
         const caseNames = args.length ? args : cases.cases;
 
-        for (let c of caseNames) {
+        for (const c of caseNames) {
           if (c === 'spectypes') {
             // hack: manually run the spectypes compilation step - avoids
             // having to run it before any other benchmark, esp when working
@@ -48,7 +48,7 @@ async function main() {
       {
         const caseNames = args as cases.CaseName[];
 
-        for (let c of caseNames) {
+        for (const c of caseNames) {
           console.log('Loading "%s"', c);
 
           await cases.importCase(c);
@@ -60,6 +60,8 @@ async function main() {
 
     default:
       console.error('unknown command:', command);
+
+      // eslint-disable-next-line no-process-exit
       process.exit(1);
   }
 }

test/benchmarks.test.ts

@@ -35,8 +35,8 @@ import '../cases/zod';
 test('all cases must have been imported in tests', () => {
   const registeredCases = new Set<string>();
 
-  getRegisteredBenchmarks().forEach(([benchmarkId, benchmarkCases]) => {
-    benchmarkCases.forEach(b => {
+  getRegisteredBenchmarks().forEach(nameBenchmarkPair => {
+    nameBenchmarkPair[1].forEach(b => {
       registeredCases.add(b.moduleName);
     });
   });