ci: simple-git-hooks fix

motss · Feb 6, 2022 · 3956fa2 · 3956fa2
1 parent 2d5e2dc
commit 3956fa2
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 19 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -81,7 +81,6 @@
     "eslint-plugin-regexp": "^1.5.1",
     "eslint-plugin-simple-import-sort": "^7.0.0",
     "nano-staged": "^0.5.0",
-    "simple-git-hooks": "^2.7.0",
     "typescript": "^4.5.5",
     "uvu": "^0.5.3",
     "watchlist": "^0.3.0"

diff --git a/postinstall.mjs b/postinstall.mjs
@@ -1,7 +1,7 @@
 #!/usr/bin/env zx
 
 const {
-  CI = false,
+  CI = 'false',
   INIT_CWD = '',
 } = process.env;
 
@@ -14,7 +14,14 @@ if (CI !== 'true') {
     !INIT_CWD.endsWith(`node_modules/${moduleName}`) &&
     INIT_CWD.endsWith(moduleName)
   ) {
+    /**
+     * NOTE: To skip running `simple-git-hooks` in CI environment.
+     * But `npm x -y -- simple-git-hooks@latest` does not work as expected so splitting it into
+     * a 2-step process: install without saving as dependency then execute it.
+     */
+    await $`npm i --no-save simple-git-hooks`
     await $`simple-git-hooks`;
+
     await $`npm dedupe`;
   }
 }