diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index d17e16da6ca5f..692d6b83a9408 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -16,11 +16,11 @@ jobs: strategy: matrix: - # Test the latest version of Node.js plus the last two LTS versions. node-version: - - "*" - - lts/* - - lts/-1 + - "19" + - "18" + - "16" + - "14" bundle: - "true" include: diff --git a/src/compiler/watch.ts b/src/compiler/watch.ts index 0d32009aee680..db7527fa004ea 100644 --- a/src/compiler/watch.ts +++ b/src/compiler/watch.ts @@ -172,7 +172,12 @@ function getPlainDiagnosticFollowingNewLines(diagnostic: Diagnostic, newLine: st export function getLocaleTimeString(system: System) { return !system.now ? new Date().toLocaleTimeString() : - system.now().toLocaleTimeString("en-US", { timeZone: "UTC" }); + // On some systems / builds of Node, there's a non-breaking space between the time and AM/PM. + // This branch is solely for testing, so just switch it to a normal space for baseline stability. + // See: + // - https://github.com/nodejs/node/issues/45171 + // - https://github.com/nodejs/node/issues/45753 + system.now().toLocaleTimeString("en-US", { timeZone: "UTC" }).replace("\u202f", " "); } /**