-
-
Notifications
You must be signed in to change notification settings - Fork 2
Sign APA repository with multiple keys at once #19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughThe GitHub Actions workflow for the "merge" job was updated to import two separate GPG keys from secrets and conditionally include them based on environment variables. A combined 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (3)
✨ Finishing Touches🧪 Generate Unit Tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
.github/workflows/build.yml (2)
83-92
: Build multi-key GPG parameters
The script concatenates--yes --armor
with-u <fingerprint>
flags for each imported key. Consider adding--batch --pinentry-mode loopback
to theGPG_PARAMETERS
string to suppress any interactive prompts in CI and ensure fully non-interactive signing.
102-109
: Use combined GPG parameters for signing
The signing loop leverages the generatedGPG_PARAMETERS
for both clear-signed and detached signatures. As an alternative, you could reference the runtime variable directly ($GPG_PARAMETERS
) instead of the expression syntax (${{ env.GPG_PARAMETERS }}
) for clarity in the shell script.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/build.yml
(3 hunks)
🔇 Additional comments (3)
.github/workflows/build.yml (3)
50-52
: Define dual GPG key env variables
ThePRIMARY_KEY
andSECONDARY_KEY
environment variables are correctly set from secrets for dual-key signing during the transition period.
71-76
: Conditional import of the primary GPG key
This step properly usesif: env.PRIMARY_KEY != ''
and thecrazy-max/ghaction-import-gpg
action to import the primary key only when provided.
77-82
: Conditional import of the secondary GPG key
Likewise, the secondary key import is gated byif: env.SECONDARY_KEY != ''
and correctly configured.
Very well done, Igor. Thank you for taking care of this. |
Sign with old and new key.