Skip to content

ci(release): add macos code signing and notarization - #1483

Merged
lukehinds merged 1 commit into
mainfrom
notary-build
Jul 24, 2026
Merged

ci(release): add macos code signing and notarization#1483
lukehinds merged 1 commit into
mainfrom
notary-build

Conversation

@lukehinds

@lukehinds lukehinds commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

This commit introduces the necessary steps for macOS code signing and notarization within the release workflow.

  • The nono binary is now signed with an Apple Developer ID certificate.
  • The signed binary is submitted to Apple's Notarization Service. This is a requirement for distributing macOS software outside the Mac App Store, allowing it to pass Gatekeeper checks without security warnings.
  • A temporary keychain is created for certificate import and is securely deleted upon completion, even if subsequent steps fail.
  • The codesign and notarytool utilities are used to perform these operations.

Linked Issue #1224

@panga 👀

This commit introduces the necessary steps for macOS code signing and notarization within the release workflow.

- The `nono` binary is now signed with an Apple Developer ID certificate.
- The signed binary is submitted to Apple's Notarization Service. This is a requirement for distributing macOS software outside the Mac App Store, allowing it to pass Gatekeeper checks without security warnings.
- A temporary keychain is created for certificate import and is securely deleted upon completion, even if subsequent steps fail.
- The `codesign` and `notarytool` utilities are used to perform these operations.

Signed-off-by: Luke Hinds <lukehinds@gmail.com>
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@github-actions github-actions Bot added size/medium ci CI/CD changes labels Jul 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR Review Summary

Size

Metric Value
Lines added +84
Lines removed -0
Total changed 84
Classification Medium (50–300 lines)

Affected crates

No crate source directories are directly affected.

Blast radius — Contained

This PR touches: CI / build tooling


Updated automatically on each push to this PR.

@nogent-nolabs-ai nogent-nolabs-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nogent code review

1 low-severity security finding.

Automated code + security review. CI already covers clippy, rustfmt, tests, cargo-audit and commit-lint.

Comment thread .github/workflows/release.yml

@connrg connrg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@lukehinds
lukehinds merged commit ac19228 into main Jul 24, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI/CD changes size/medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants