From 99d33e3df7ebfb2973d4f8e544dde5f4fe296945 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jakub=20Ber=C3=A1nek?= <berykubik@gmail.com>
Date: Thu, 7 Dec 2023 15:02:08 +0100
Subject: [PATCH] Deny unused variables and parameters in TypeScript/Vue

---
 site/frontend/src/graph/render.ts                      | 10 +++++-----
 site/frontend/src/pages/bootstrap/plots.ts             |  4 ++--
 .../pages/compare/compile/table/comparisons-table.vue  |  1 -
 site/frontend/src/pages/compare/metric-selector.vue    |  1 -
 site/frontend/src/pages/compare/summary/count.vue      |  2 +-
 site/frontend/src/pages/compare/tabs.vue               |  2 +-
 site/frontend/src/pages/status/page.vue                |  4 +---
 site/frontend/tsconfig.json                            |  4 +++-
 8 files changed, 13 insertions(+), 15 deletions(-)

diff --git a/site/frontend/src/graph/render.ts b/site/frontend/src/graph/render.ts
index 1dd1ef6d4..36a3ccb14 100644
--- a/site/frontend/src/graph/render.ts
+++ b/site/frontend/src/graph/render.ts
@@ -210,7 +210,7 @@ function wheelZoomPlugin(opts) {
               });
             }
 
-            function onup(e) {
+            function onup(_e) {
               document.removeEventListener("mousemove", onmove);
               document.removeEventListener("mouseup", onup);
             }
@@ -294,7 +294,7 @@ function genPlotOpts({
     },
     scales: {
       y: {
-        range: (self, dataMin, dataMax) =>
+        range: (_self, dataMin, dataMax) =>
           uPlot.rangeNum(absoluteMode ? 0 : dataMin, dataMax, 0.2, true),
       },
     },
@@ -307,7 +307,7 @@ function genPlotOpts({
       {
         label: yAxisLabel,
         space: 24,
-        values: (self, splits) => {
+        values: (_self, splits) => {
           return splits.map((v) => {
             return v >= 1e12
               ? v / 1e12 + "T"
@@ -328,7 +328,7 @@ function genPlotOpts({
           drawAxes: [
             (u) => {
               let {ctx} = u;
-              let {left, top, width, height} = u.bbox;
+              let {top, height} = u.bbox;
 
               const interpolatedColorWithAlpha = "#fcb0f15f";
 
@@ -355,7 +355,7 @@ function genPlotOpts({
         },
       },
       tooltipPlugin({
-        onclick(u, seriesIdx, dataIdx) {
+        onclick(_u, _seriesIdx, dataIdx) {
           let thisCommit = commits[dataIdx][1];
           let prevCommit = (commits[dataIdx - 1] || [null, null])[1];
           window.open(
diff --git a/site/frontend/src/pages/bootstrap/plots.ts b/site/frontend/src/pages/bootstrap/plots.ts
index e7cce2849..b809a2691 100644
--- a/site/frontend/src/pages/bootstrap/plots.ts
+++ b/site/frontend/src/pages/bootstrap/plots.ts
@@ -140,14 +140,14 @@ function genPlotOpts({
       {
         label: yAxisLabel,
         space: 24,
-        values: (self, splits) => {
+        values: (_self, splits) => {
           return splits.map((v) => v / 1e9 + " sec");
         },
       },
     ],
     plugins: [
       tooltipPlugin({
-        onclick(u, seriesIdx, dataIdx) {
+        onclick(_u, _seriesIdx, dataIdx) {
           let thisCommit = commits[dataIdx][1];
           let prevCommit = (commits[dataIdx - 1] || [null, null])[1];
           window.open(`/compare.html?start=${prevCommit}&end=${thisCommit}`);
diff --git a/site/frontend/src/pages/compare/compile/table/comparisons-table.vue b/site/frontend/src/pages/compare/compile/table/comparisons-table.vue
index b1e97a0bb..945d5177f 100644
--- a/site/frontend/src/pages/compare/compile/table/comparisons-table.vue
+++ b/site/frontend/src/pages/compare/compile/table/comparisons-table.vue
@@ -7,7 +7,6 @@ import {CompileBenchmarkMap, CompileTestCase} from "../common";
 import {computed} from "vue";
 import {useExpandedStore} from "./expansion";
 import BenchmarkDetail from "./benchmark-detail.vue";
-import {getPastDate} from "./utils";
 
 const props = defineProps<{
   id: string;
diff --git a/site/frontend/src/pages/compare/metric-selector.vue b/site/frontend/src/pages/compare/metric-selector.vue
index 5b5d94ab1..e15040bcb 100644
--- a/site/frontend/src/pages/compare/metric-selector.vue
+++ b/site/frontend/src/pages/compare/metric-selector.vue
@@ -3,7 +3,6 @@ import {
   createUrlWithAppendedParams,
   navigateToUrlParams,
 } from "../../utils/navigation";
-import {BenchmarkInfo} from "../../api";
 import {MetricDescription} from "./metrics";
 
 const props = defineProps<{
diff --git a/site/frontend/src/pages/compare/summary/count.vue b/site/frontend/src/pages/compare/summary/count.vue
index 5faddfc4d..b52a863e6 100644
--- a/site/frontend/src/pages/compare/summary/count.vue
+++ b/site/frontend/src/pages/compare/summary/count.vue
@@ -7,6 +7,6 @@ const props = defineProps<{
 
 <template>
   <span :title="cases + ' test case(s), ' + benchmarks + ' unique benchmark(s)'"
-    >{{ cases }} ({{ benchmarks }})</span
+    >{{ props.cases }} ({{ props.benchmarks }})</span
   >
 </template>
diff --git a/site/frontend/src/pages/compare/tabs.vue b/site/frontend/src/pages/compare/tabs.vue
index 7008ae52d..6b711ee29 100644
--- a/site/frontend/src/pages/compare/tabs.vue
+++ b/site/frontend/src/pages/compare/tabs.vue
@@ -1,5 +1,5 @@
 <script setup lang="tsx">
-import {computed, h, ref, Ref} from "vue";
+import {h, ref, Ref} from "vue";
 import {CompareResponse, Tab} from "./types";
 import {
   diffClass,
diff --git a/site/frontend/src/pages/status/page.vue b/site/frontend/src/pages/status/page.vue
index bd39e03e8..7e8d0ad35 100644
--- a/site/frontend/src/pages/status/page.vue
+++ b/site/frontend/src/pages/status/page.vue
@@ -2,7 +2,7 @@
 import {getJson} from "../../utils/requests";
 import {STATUS_DATA_URL} from "../../urls";
 import {withLoading} from "../../utils/loading";
-import {computed, ref, Ref, watch} from "vue";
+import {computed, ref, Ref} from "vue";
 import {
   Artifact,
   BenchmarkError,
@@ -16,10 +16,8 @@ import {
   differenceInSeconds,
   format,
   fromUnixTime,
-  sub,
   subSeconds,
 } from "date-fns";
-import {CompileTestCase} from "../compare/compile/common";
 import {useExpandedStore} from "./expansion";
 
 async function loadStatus(loading: Ref<boolean>) {
diff --git a/site/frontend/tsconfig.json b/site/frontend/tsconfig.json
index 7056a0bc4..20a409046 100644
--- a/site/frontend/tsconfig.json
+++ b/site/frontend/tsconfig.json
@@ -7,6 +7,8 @@
     "rootDir": "src",
     "esModuleInterop": true,
     "jsx": "preserve",
-    "jsxFactory": "h"
+    "jsxFactory": "h",
+    "noUnusedLocals": true,
+    "noUnusedParameters": true
   }
 }