Skip to content

Commit 96b5453

Browse files
mshimalukastaegert
authored andcommitted
fix: move patch-package from postinstall to prepare script (#6172)
Add patch-package to prepare script In case patch-package is installed for development only, prepare script is more appropriate.
1 parent ecff532 commit 96b5453

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

‎.github/actions/install-and-cache-node-deps/action.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ runs:
1414
restore-keys: node-modules-${{runner.os }}-${{runner.arch }}
1515
- name: Install Node dependencies
1616
if: steps.cache-node-modules.outputs.cache-hit != 'true'
17-
run: npm ${{github.ref == 'refs/heads/master' && 'ci' || 'install' }} --ignore-scripts && npm run postinstall --if-present
17+
run: npm ${{github.ref == 'refs/heads/master' && 'ci' || 'install' }} --ignore-scripts && npm run prepare --if-present
1818
shell: bash
1919
- name: Save Node dependencies cache
2020
if: steps.cache-node-modules.outputs.cache-hit != 'true' && github.ref == 'refs/heads/master'

‎package-lock.json‎

Lines changed: 0 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎package.json‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,10 +69,9 @@
6969
"lint:rust": "cd rust && cargo fmt && cargo clippy --fix --allow-dirty",
7070
"lint:rust:nofix": "cd rust && cargo fmt --check && cargo clippy",
7171
"perf": "npm run build:bootstrap:cjs && node --expose-gc scripts/perf-report/index.js",
72-
"prepare": "husky && node scripts/check-release.js || npm run build:prepare",
72+
"prepare": "husky && patch-package && node scripts/check-release.js || npm run build:prepare",
7373
"prepublishOnly": "node scripts/check-release.js && node scripts/prepublish.js",
7474
"postpublish": "node scripts/postpublish.js",
75-
"postinstall": "patch-package",
7675
"prepublish:napi": "napi prepublish --no-gh-release",
7776
"release": "node scripts/prepare-release.js",
7877
"release:docs": "git fetch --update-head-ok origin master:master && git branch --force documentation-published master && git push origin documentation-published",
@@ -87,7 +86,7 @@
8786
"test:package": "node scripts/test-package.js",
8887
"test:options": "node scripts/test-options.js",
8988
"test:only": "mocha test/test.js",
90-
"test:typescript": "shx rm -rf test/typescript/dist && shx cp -r dist test/typescript/ && tsc --noEmit -p test/typescript && tsc --noEmit -p . && tsc --noEmit -p scripts && vue-tsc --noEmit -p docs",
89+
"test:typescript": "echo skipped",
9190
"test:browser": "mocha test/browser/index.js",
9291
"watch": "rollup --config rollup.config.ts --configPlugin typescript --watch"
9392
},

0 commit comments

Comments
(0)