Read the signing identity and team from repository variables

[agoodkind]

Co-authored-by: Claude noreply@anthropic.com

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[agoodkind]

• Read the signing identity and team from repository variables #112 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: QUIET

Plan: Pro Plus

Run ID: c3d9cc19-f3dc-4484-a70f-95479a75b3f6

📥 Commits

Reviewing files that changed from the base of the PR and between 8848277 and c7193d2.

📒 Files selected for processing (2)

• .github/workflows/ci.yml
• .github/workflows/release.yml

---

📝 Walkthrough

Walkthrough

The CI and release workflows now source Apple signing identities and the Apple team ID from repository variables. The release workflow also documents the distinction between Developer ID and Apple Distribution identities.

Changes

Apple signing configuration

Layer / File(s)	Summary
Workflow signing variables .github/workflows/ci.yml, .github/workflows/release.yml	The workflows use repository variables for Apple signing identities and the Apple team ID. Release comments distinguish Developer ID from Apple Distribution identities.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

• agoodkind/iphone-cell-tunnel#111: Updates Apple signing certificate configuration in the CI and release workflows.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description only provides a co-authorship attribution and does not describe the workflow changes.	Add a brief summary that the CI and release workflows now read the signing identity and Apple team ID from repository variables.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly states that the signing identity and team are now read from repository variables.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch signing-identity-to-ci-vars

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}