chore: 🤖 explicitly require managing pnpm "built dependencies" (#2873)
pnpm by default refuses to run install scripts, some packages require these and others are use install scripts as a way to provide messages. Instead of a pnpm warning this commit ensures that the package that is detected with an install script must be either listed in ignoredBuiltDependencies or onlyBuiltDependencies. Running pnpm install will fail until a package has been categorized As part of this commit the packages that were previously listed as warnings have been categorized under ignoredBuiltDependencies since their install scripts are not required for the functionality of the package
This commit is contained in:
1 parent
beeee851ac
commit
f738e71657
4 files changed
+12
-3
No files matched your search
+8
-2
@@ -48,7 +48,13 @@
|
|||||||
"node-gyp": "^10.0.0",
|
"node-gyp": "^10.0.0",
|
||||||
"ansi-html": "^0.0.9",
|
"ansi-html": "^0.0.9",
|
||||||
"fireworm>async": "^2.6.4"
|
"fireworm>async": "^2.6.4"
|
||||||
}
|
},
|
||||||
|
"ignoredBuiltDependencies": [
|
||||||
|
"@parcel/watcher",
|
||||||
|
"core-js",
|
||||||
|
"tesseract.js"
|
||||||
|
],
|
||||||
|
"onlyBuiltDependencies": []
|
||||||
},
|
},
|
||||||
"config": {
|
"config": {
|
||||||
"commitizen": {
|
"commitizen": {
|
||||||
@@ -59,4 +65,4 @@
|
|||||||
"node": "20.* || 22.*"
|
"node": "20.* || 22.*"
|
||||||
},
|
},
|
||||||
"packageManager": "pnpm@10.8.1"
|
"packageManager": "pnpm@10.8.1"
|
||||||
}
|
}
|
||||||
@@ -1,3 +1,4 @@
|
|||||||
|
strictDepBuilds: true
|
||||||
packages:
|
packages:
|
||||||
- addons/*
|
- addons/*
|
||||||
- ui/*
|
- ui/*
|
||||||
|
|||||||
@@ -44,6 +44,7 @@
|
|||||||
"unzipper": "^0.11.5"
|
"unzipper": "^0.11.5"
|
||||||
},
|
},
|
||||||
"pnpm": {
|
"pnpm": {
|
||||||
|
"ignoredBuiltDependencies": [],
|
||||||
"onlyBuiltDependencies": [
|
"onlyBuiltDependencies": [
|
||||||
"electron",
|
"electron",
|
||||||
"electron-winstaller",
|
"electron-winstaller",
|
||||||
|
|||||||
@@ -1,3 +1,4 @@
|
|||||||
|
strictDepBuilds: true
|
||||||
|
nodeLinker: hoisted
|
||||||
packages:
|
packages:
|
||||||
- .
|
- .
|
||||||
nodeLinker: hoisted
|
|
||||||
Reference in new issue
Block a user