Type-check transform files

Author: gitKrystan

.github/workflows/tests.yml

@@ -10,10 +10,10 @@ jobs:
     timeout-minutes: 15
 
     steps:
-    - uses: actions/checkout@v1
-    - uses: actions/setup-node@v1
+    - uses: actions/checkout@v3
+    - uses: actions/setup-node@v3
       with:
-        node-version: '10.x'
+        node-version: '16.x'
     - name: Install
       run: yarn install
     - name: Linting
@@ -30,10 +30,10 @@ jobs:
     timeout-minutes: 15
 
     steps:
-    - uses: actions/checkout@v1
-    - uses: actions/setup-node@v1
+    - uses: actions/checkout@v3
+    - uses: actions/setup-node@v3
       with:
-        node-version: '10.x'
+        node-version: '16.x'
     - name: Install
       run: yarn install
     - name: Test

package.json

@@ -14,11 +14,16 @@
   "author": "",
   "scripts": {
     "release": "release-it",
+    "prepublishOnly": "yarn build",
+    "postpublish": "yarn clean",
+    "build": "tsc",
+    "clean": "tsc --build --clean",
     "test": "jest",
     "test:integration": "ts-node ./test/run-test.ts",
     "update-docs": "codemod-cli update-docs",
     "coveralls": "cat ./coverage/lcov.info | node node_modules/.bin/coveralls",
     "lint:js": "eslint .",
+    "lint:ts": "tsc --noEmit",
     "debug:telemetry": "node --inspect-brk ./bin/telemetry.js",
     "debug:codemod:not-working": "node --inspect-brk ./bin/cli",
     "codemod": "jscodeshift -t ./transforms/no-implicit-this/index.js --extensions js,ts,hbs --run-in-band",
@@ -37,10 +42,13 @@
     "ember-template-recast": "^3.3.2"
   },
   "devDependencies": {
+    "@tsconfig/node16": "^1.0.4",
+    "@tsconfig/strictest": "^2.0.1",
     "@types/chalk": "^2.2.0",
     "@types/common-tags": "^1.8.0",
-    "@typescript-eslint/eslint-plugin": "^2.34.0",
-    "@typescript-eslint/parser": "^2.34.0",
+    "@types/node": "^20.2.3",
+    "@typescript-eslint/eslint-plugin": "^5.59.7",
+    "@typescript-eslint/parser": "^5.59.7",
     "babel-plugin-htmlbars-inline-precompile": "^3.0.1",
     "chalk": "^4.1.1",
     "common-tags": "^1.8.0",
@@ -53,11 +61,11 @@
     "prettier": "^1.19.1",
     "release-it": "^14.6.2",
     "release-it-lerna-changelog": "^3.1.0",
-    "ts-node": "^8.10.2",
-    "typescript": "~3.9.7"
+    "ts-node": "^10.9.1",
+    "typescript": "~5.0.4"
   },
   "engines": {
-    "node": "8.* || 10.* || >= 12"
+    "node": "16.* || 18.* || >= 20"
   },
   "jest": {
     "testEnvironment": "node",

test/run-test.ts

@@ -7,7 +7,7 @@ import { runTestIntegrationSequence } from './helpers/sequence';
 
 const allVersions = ['3.10', '3.13'];
 
-(async () => {
+(async (): Promise<void> => {
   const emberVersion = process.env.EMBER_VERSION;
 
   if (!emberVersion) {
@@ -31,7 +31,7 @@ const allVersions = ['3.10', '3.13'];
 
     didSucceed = false;
   } finally {
-    // TOOD: if there were any changes to the fixtures directories, revert them
+    // TODO: if there were any changes to the fixtures directories, revert them
     try {
       // const fixturePath = path.join(process.cwd(), 'test', 'fixtures', emberVersion);
       // await execa(`git checkout -- .`, { cwd: fixturePath });
@@ -44,4 +44,7 @@ const allVersions = ['3.10', '3.13'];
   }
 
   process.exit(didSucceed ? 0 : 1);
-})();
+})().catch(e => {
+  console.error(e);
+  process.exit(1);
+});

tsconfig.json

@@ -1,17 +1,13 @@
 {
+  "extends": ["@tsconfig/node16/tsconfig.json", "@tsconfig/strictest/tsconfig.json"],
   "compilerOptions": {
-    "module": "commonjs",
-    "esModuleInterop": true,
-    "allowSyntheticDefaultImports": true,
-    "target": "es6",
-    "noImplicitAny": true,
-    "moduleResolution": "node",
     "sourceMap": true,
+    // FIXME: "skipLibCheck": false,
     "outDir": "dist",
     "baseUrl": ".",
     "paths": {
-      "*": ["node_modules/*", "types/*"]
+      "*": ["types/*"]
     }
   },
-  "include": ["test/**/*"]
+  "include": ["transforms/**/*", "test/**/*"]
 }