Skip to content

Preserve npm separator for verified Electron packaging - #23

Merged
bubblegump30 merged 1 commit into
mainfrom
release/v2.6.1-source
Nov 8, 2026
Merged

bubblegump30 merged 1 commit into
mainfrom
release/v2.6.1-source

Conversation

@bubblegump30

Copy link
Copy Markdown
Owner

Fixes PowerShell consuming the static -- separator before it reached npm. Passes an explicit argument array to the existing wrapper so the verified Electron distribution option reaches electron-builder. A real temporary npm script received the exact --config.electronDist argument through the actual patched wrapper. Application/UI files are unchanged. Cancels the in-progress release run and rebuilds only after this correction passes CI.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 8, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-08T09:48:47.815608Z fb4131f PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@bubblegump30
bubblegump30 merged commit 785fcac into main Nov 8, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant