Skip to content

Commit 4d4cb9f

Browse files
steveseguinclaude
andcommitted
Fix afterSign.js to properly skip on non-macOS platforms
- Check electronPlatformName instead of process.platform for cross-platform builds - Prevents error when building Windows on macOS - Version bump to 2.20.0 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 8a85d67 commit 4d4cb9f

3 files changed

Lines changed: 434 additions & 104 deletions

File tree

afterSign.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,13 @@ const path = require('path');
33
var electron_notarize = require('electron-notarize');
44

55
module.exports = async function (params) {
6+
console.log('afterSign hook triggered', params);
7+
68
// Only notarize the app on Mac OS only.
7-
if (process.platform !== 'darwin') {
9+
if (params.electronPlatformName !== 'darwin') {
10+
console.log('Skipping notarization - not macOS');
811
return;
912
}
10-
console.log('afterSign hook triggered', params);
1113

1214
// Same appId in electron-builder.
1315
let appId = 'ninja.vdo'

0 commit comments

Comments
 (0)