chore: remove unnecessary eslint override

[Cangit]

in decorate.js

Checklist

• run npm run test and npm run benchmark
• tests and/or benchmarks are included
• documentation is changed or added
• commit message and code follows the Developer's Certification of Origin
  and the Code of conduct

[gurgunday]

Btw I no longer measure a difference between var and let, but don’t see why that change was combined with this PR

[Cangit]

Sorry, should have made that change more clear. But yes, var should be unnecessary, and that in turn makes the eslint-disable-next-line no-var unnecessary.

[gurgunday]

If anyone wants to bench it:

function benchmarkVarLoop(iterations) {
  let sum = 0;
  const startTime = performance.now();
  for (var i = 0; i < iterations; i++) {
    sum += i;
  }
  const endTime = performance.now();
  return endTime - startTime;
}

function benchmarkLetLoop(iterations) {
  let sum = 0;
  const startTime = performance.now();
  for (let i = 0; i < iterations; i++) {
    sum += i;
  }
  const endTime = performance.now();
  return endTime - startTime;
}

function runBenchmarks() {
  const iterations = 1000000;

  const varDuration = benchmarkVarLoop(iterations);
  console.log(`var loop duration: ${varDuration} ms`);

  const letDuration = benchmarkLetLoop(iterations);
  console.log(`let loop duration: ${letDuration} ms`);

  if (varDuration < letDuration) {
    console.log("var is faster than let in this benchmark.");
  } else if (varDuration > letDuration) {
    console.log("let is faster than var in this benchmark.");
  } else {
    console.log("var and let have the same performance in this benchmark.");
  }
}

runBenchmarks();