dx: Oxlint warns for unused ignores (#2229)

Author: aleksanderkatan

apps/infra-benchmarks/src/discriminated-union.ts

@@ -10,7 +10,7 @@ const bench = new Bench({
   name: 'discriminated union',
   time: 100,
   async setup() {
-    // oxlint-disable-next-line typescript/no-explicit-any making sure GC has no impact on the results
+    // oxlint-disable-next-line typescript/no-explicit-any -- making sure GC has no impact on the results
     (globalThis as any).gc();
   },
 });

apps/treeshake-test/compareResults.ts

@@ -19,7 +19,7 @@ function groupResultsByTest(results: typeof BenchmarkResults.infer) {
     if (!grouped[result.testFilename]) {
       grouped[result.testFilename] = {};
     }
-    // oxlint-disable-next-line typescript/no-non-null-assertion it's there...
+    // oxlint-disable-next-line typescript/no-non-null-assertion -- it's there...
     grouped[result.testFilename]![result.bundler] = result.size;
   }
   return grouped;

apps/typegpu-docs/src/examples/algorithms/mnist-inference/index.ts

@@ -208,7 +208,7 @@ const network = createNetwork(await downloadLayers(root));
 const canvas = document.querySelector('canvas') as HTMLCanvasElement;
 const context = canvas.getContext('2d') as CanvasRenderingContext2D;
 
-// oxlint-disable-next-line typescript/no-unnecessary-type-assertion not really unnecessary
+// oxlint-disable-next-line typescript/no-unnecessary-type-assertion -- not really unnecessary
 const bars = Array.from(document.querySelectorAll('.bar')) as HTMLDivElement[];
 const subgroupsEl = document.getElementById(
   'subgroups-status',

apps/typegpu-docs/src/examples/algorithms/probability/executor.ts

@@ -104,7 +104,7 @@ export class Executor {
       this.#pipelineCache.set(distribution, pipeline);
     }
 
-    // oxlint-disable-next-line typescript/no-non-null-assertion just checked it above
+    // oxlint-disable-next-line typescript/no-non-null-assertion -- just checked it above
     return this.#pipelineCache.get(distribution)!;
   }
 

apps/typegpu-docs/src/examples/exampleContent.ts

@@ -1,4 +1,4 @@
-// oxlint-disable typescript/no-unnecessary-type-assertion import.meta.glob is inferred incorrectly by oxlint
+// oxlint-disable typescript/no-unnecessary-type-assertion -- import.meta.glob is inferred incorrectly by oxlint
 import pathe from 'pathe';
 import * as R from 'remeda';
 import type {

apps/typegpu-docs/src/examples/image-processing/background-segmentation/index.ts

@@ -60,9 +60,9 @@ if (!device || !adapter) {
 }
 
 // monkey patching ONNX: https://github.com/microsoft/onnxruntime/issues/26107
-// oxlint-disable-next-line typescript/unbound-method we know what we're doing
+// oxlint-disable-next-line typescript/unbound-method -- we know what we're doing
 const oldRequestAdapter = navigator.gpu.requestAdapter;
-// oxlint-disable-next-line typescript/unbound-method we know what we're doing
+// oxlint-disable-next-line typescript/unbound-method -- we know what we're doing
 const oldRequestDevice = adapter.requestDevice;
 navigator.gpu.requestAdapter = async () => adapter;
 adapter.requestDevice = async () => device;

apps/typegpu-docs/src/examples/rendering/function-visualizer/index.ts

@@ -68,7 +68,7 @@ const computeLayout = tgpu.bindGroupLayout({
 
 const functionExprSlot = tgpu.slot<TgpuRawCodeSnippet<d.F32>>();
 
-// oxlint-disable-next-line no-unused-vars it is used in wgsl
+// oxlint-disable-next-line no-unused-vars -- it is used in wgsl
 const interpolatedFunction = (x: number) => {
   'use gpu';
   return functionExprSlot.$;

apps/typegpu-docs/src/examples/tests/tgsl-parsing-test/logical-expressions.ts

@@ -1,4 +1,4 @@
-// oxlint-disable typescript/no-unnecessary-boolean-literal-compare this is a test
+// oxlint-disable typescript/no-unnecessary-boolean-literal-compare -- this is a test
 import tgpu, { d, std } from 'typegpu';
 
 const Schema = d.struct({

apps/typegpu-docs/src/pages/benchmark/components/checkbox-tree.tsx

@@ -84,7 +84,7 @@ function TestCheckbox(props: { suiteName: string; testName: string }) {
   const state = selected.includes(identifier) ? 'checked' : 'unchecked';
 
   function changeState() {
-    // oxlint-disable-next-line no-unused-expressions it's a call
+    // oxlint-disable-next-line no-unused-expressions -- it's a call
     selected.includes(identifier)
       ? setSelected(selected.filter((item) => item !== identifier))
       : setSelected([...selected, identifier]);

apps/typegpu-docs/src/pages/benchmark/suites.ts

@@ -25,7 +25,7 @@ export type SuiteSetupOptions = {
   d: TypeGPUDataModule;
 };
 
-// oxlint-disable-next-line typescript/no-explicit-any <sshhhhhh...>
+// oxlint-disable-next-line typescript/no-explicit-any -- sshhhhhh...
 export type Suite<T extends { bench: Bench } = any> = {
   setup: (options: SuiteSetupOptions) => T;
   tests: Record<string, (getCtx: () => T) => () => Promise<unknown>>;