build: test on node 14, 16, 18, 20, and 21

.github/workflows/ci.yaml

@@ -1,6 +1,6 @@
 on:
   push:
-    branches: [ $default-branch ]
+    branches: [$default-branch]
   pull_request:
 name: ci
 jobs:
@@ -9,7 +9,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node: [14, 16, 18]
+        node: [14, 16, 18, 20, 21]
     steps:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
       - uses: actions/setup-node@v4
@@ -29,7 +29,7 @@ jobs:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 16
+          node-version: 20
       - run: npm ci
       - run: npm run system-test
   system_test:
@@ -38,7 +38,7 @@ jobs:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 16
+          node-version: 20
       - run: npm ci
       - run: npm run system-test
   lint:
@@ -47,7 +47,7 @@ jobs:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 16
+          node-version: 20
       - run: npm ci
       - run: npm run lint
   license_check:
@@ -56,6 +56,6 @@ jobs:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 16
+          node-version: 20
       - run: npm ci
       - run: npm run license-check

.github/workflows/release.yaml

@@ -9,7 +9,7 @@ jobs:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 16
+          node-version: 20
           registry-url: 'https://wombat-dressing-room.appspot.com'
       - run: npm ci
       - run: npm publish

package.json

@@ -34,12 +34,7 @@
   "engines": {
     "node": ">=14"
   },
-  "keywords": [
-    "typescript",
-    "linter",
-    "formatter",
-    "google"
-  ],
+  "keywords": ["typescript", "linter", "formatter", "google"],
   "author": "Google Inc.",
   "license": "Apache-2.0",
   "dependencies": {
@@ -69,7 +64,6 @@
     "@types/mocha": "^10.0.0",
     "@types/ncp": "^2.0.4",
     "@types/node": "20.11.30",
-    "@types/prettier": "^3.0.0",
     "@types/rimraf": "^3.0.0",
     "@types/sinon": "^17.0.0",
     "@types/tmp": "^0.2.0",

src/init.ts

@@ -19,12 +19,12 @@ import * as inquirer from 'inquirer';
 import * as path from 'path';
 import {ncp} from 'ncp';
 import * as util from 'util';
+import * as writeFileAtomic from 'write-file-atomic';
 
 import {
   getPkgManagerCommand,
   readFilep as read,
   readJsonp as readJson,
-  writeFileAtomicp as write,
   Bag,
   DefaultPackage,
 } from './util';
@@ -168,7 +168,7 @@ async function writePackageJson(
 ): Promise<void> {
   options.logger.log('Writing package.json...');
   if (!options.dryRun) {
-    await write('./package.json', formatJson(packageJson));
+    await writeFileAtomic('./package.json', formatJson(packageJson));
   }
   const preview = {
     scripts: packageJson.scripts,
@@ -216,7 +216,7 @@ async function generateConfigFile(
   if (writeFile) {
     options.logger.log(`Writing ${filename}...`);
     if (!options.dryRun) {
-      await write(filename, contents);
+      await writeFileAtomic(filename, contents);
     }
     options.logger.log(contents);
   }

src/util.ts

@@ -19,12 +19,10 @@ import * as path from 'path';
 import * as rimraf from 'rimraf';
 import {promisify} from 'util';
 import * as ncp from 'ncp';
-import * as writeFileAtomic from 'write-file-atomic';
 import * as JSON5 from 'json5';
 
 export const readFilep = promisify(fs.readFile);
 export const rimrafp = promisify(rimraf);
-export const writeFileAtomicp = promisify(writeFileAtomic);
 export const ncpp = promisify(ncp.ncp);
 
 export interface Bag<T> {