chore: enable ban-untagged-todo lint rule and tag existing TODOs

csv/parse.ts

@@ -240,7 +240,7 @@ class Parser {
     } else if (options.fieldsPerRecord === 0) {
       _nbFields = "UNINITIALIZED";
     } else {
-      // TODO: Should we check if it's a valid integer?
+      // TODO(magurotuna): Should we check if it's a valid integer?
       _nbFields = options.fieldsPerRecord;
     }
 

csv/parse_stream.ts

@@ -382,7 +382,7 @@ export class CsvParseStream<
     } else if (this.#options.fieldsPerRecord === 0) {
       this.#fieldsPerRecord = "UNINITIALIZED";
     } else {
-      // TODO: Should we check if it's a valid integer?
+      // TODO(magurotuna): Should we check if it's a valid integer?
       this.#fieldsPerRecord = this.#options.fieldsPerRecord;
     }
 

deno.json

@@ -50,6 +50,7 @@
     "rules": {
       "tags": ["recommended", "jsr"],
       "include": [
+        "ban-untagged-todo",
         "camelcase",
         "no-import-prefix",
         "no-sync-fn-in-async-fn",

dotenv/mod_test.ts

@@ -207,7 +207,7 @@ Deno.test(
   },
 );
 
-//TODO test permissions
+// TODO(cknight): test permissions
 
 Deno.test(
   "loadSync() prevents file system reads of default path parameter values by using explicit null",

testing/_snapshot_utils.ts

@@ -105,5 +105,5 @@ export function getSnapshotNotMatchMessage(
   return getErrorMessage(message, options);
 }
 
-// TODO (WWRS): Remove this when we drop support for Deno 1.x
+// TODO(WWRS): Remove this when we drop support for Deno 1.x
 export const LINT_SUPPORTED = !Deno.version.deno.startsWith("1.");

toml/_test_utils.ts

@@ -34,7 +34,7 @@ export function convertTestCase(v: TestCase): unknown {
         return parseFloat(v.value.replace("inf", "Infinity"));
       case "bool":
         return v.value === "true";
-      // TODO: https://github.com/denoland/std/issues/6591
+      // TODO(4513ECHO): https://github.com/denoland/std/issues/6591
       case "datetime":
       case "datetime-local":
       case "date-local":