c Downgrades FIXMEs that don't need fixing yet to TODOs

quatico-solutions · Apr 15, 2024 · 4b21959 · 4b21959
1 parent 881844d
commit 4b21959
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/packages/compiler/src/command.spec.ts b/packages/compiler/src/command.spec.ts
@@ -193,7 +193,7 @@ describe("addCompileCommand", () => {
         expect(target.getOptions().debug).toBe(true);
     });
 
-    // FIXME: This test is failing because the compiler watch support is broken.
+    // TODO: This test is failing because the compiler watch support is broken.
     it.skip("should set watch compiler option w/ --watch cli argument", () => {
         const target = new Compiler(createOptions({}, new NoReporter()));
 
@@ -220,7 +220,7 @@ describe("addCompileCommand", () => {
         expect(target.getOptions().transpileOnly).toBe(true);
     });
 
-    // FIXME: This test requires a decision regarding our explicit logic for additional cmdline arguments - do provide them through the CompilerOptions or do we reject them.
+    // TODO: This test requires a decision regarding our explicit logic for additional cmdline arguments - do provide them through the CompilerOptions or do we reject them.
     it.skip("should inform CLI user about tsconfig usage for unsupported command line arguments", () => {
         console.error = line => {
             throw new Error(line.toString());

diff --git a/packages/compiler/src/options.ts b/packages/compiler/src/options.ts
@@ -12,7 +12,7 @@ import {
     resolveCompilationConfig,
     resolveProjectConfig as resolveTsConfig,
     resolveTargets,
-    updateCompilerOptions
+    updateCompilerOptions,
 } from "@quatico/websmith-core";
 import { dirname, join } from "path";
 import ts from "typescript";
@@ -66,7 +66,7 @@ export const createOptions = (args: CompilerArguments, reporter: Reporter = new
         buildDir: args.buildDir ?? system.getCurrentDirectory(),
         config: compilationConfig,
         debug: args.debug ?? DEFAULTS.debug,
-        // FIXME: Do we need lib files, or is injecting them into the system sufficient?
+        // TODO: Do we need lib files, or is injecting them into the system sufficient?
         // files?: Record<string, string>;
         tsconfig,
         project: tsconfig.options,

diff --git a/packages/core/src/compiler/Compiler.spec.ts b/packages/core/src/compiler/Compiler.spec.ts
@@ -671,7 +671,7 @@ describe("report", () => {
     });
 });
 
-// FIXME: Update with BrowserSystem testSystem when BrowserSystem supports watch
+// TODO: Update with BrowserSystem testSystem when BrowserSystem supports watch
 describe("watch", () => {
     let testObj: Compiler;