Skip to content

Commit d8b3152

Browse files
mschileclaude
andcommitted
chore: tell knip @electron/get is dynamically required
The health-check job flagged @electron/get as an unused devDependency in packages/electron. It is used, but through `require(`@${e}/get`)` — the same computed-string form already used for @electron/packager, which keeps the module undiscoverable by mksnapshot since it is build-only. knip cannot resolve that statically. Add it alongside the existing @electron/packager entry in the package's ignoreDependencies rather than making the require a plain literal, which would expose a build dependency to the v8 snapshot. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 parent 288b5b3 commit d8b3152

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

knip.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -375,6 +375,7 @@
375375
"src/**/*.{ts,tsx,js,jsx}"
376376
],
377377
"ignoreDependencies": [
378+
"@electron/get",
378379
"@electron/packager"
379380
]
380381
},

0 commit comments

Comments
 (0)