Merge pull request #3790 from iclanton/remove-postinstall

[jest-heft-plugin] Remove the Jest postinstall step.
microsoft · Nov 29, 2022 · 0a46e7e · 0a46e7e
2 parents db2e737 + fe5ce8c
commit 0a46e7e
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 109 deletions.
diff --git a/common/changes/@rushstack/heft-jest-plugin/remove-postinstall_2022-11-28-23-57.json b/common/changes/@rushstack/heft-jest-plugin/remove-postinstall_2022-11-28-23-57.json
@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "@rushstack/heft-jest-plugin",
+      "comment": "Remove a postinstall step that patches Jest in-place. This is better achieved with a PNPM patch. See https://github.com/microsoft/rushstack/pull/3790 for more information.",
+      "type": "minor"
+    }
+  ],
+  "packageName": "@rushstack/heft-jest-plugin"
+}
diff --git a/heft-plugins/heft-jest-plugin/includes/jestScriptTransformerPatch.js b/heft-plugins/heft-jest-plugin/includes/jestScriptTransformerPatch.js
diff --git a/heft-plugins/heft-jest-plugin/package.json b/heft-plugins/heft-jest-plugin/package.json
@@ -15,8 +15,7 @@
     "build": "heft build --clean",
     "start": "heft test --clean --watch",
     "_phase:build": "heft build --clean",
-    "_phase:test": "heft test --no-build",
-    "postinstall": "node ./includes/jestScriptTransformerPatch.js"
+    "_phase:test": "heft test --no-build"
   },
   "peerDependencies": {
     "@rushstack/heft": "^0.48.8"