Skip to content

Conversation

@NormB
Copy link
Owner

@NormB NormB commented Nov 20, 2025

Summary

Fixes the wiki-sync automation failure caused by repository rule requiring verified commit signatures.

Problem

The wiki-sync workflow was failing with:

remote: error: GH013: Repository rule violations found for refs/heads/wiki-sync-auto
remote: - Commits must have verified signatures.

The peter-evans/create-pull-request@v6 action was creating unsigned commits, which violated the repository's branch protection rules.

Solution

Added GPG commit signing to the wiki-sync workflow:

  • Import GPG key using crazy-max/ghaction-import-gpg@v6 action
  • Configure git to sign commits automatically
  • Added signoff: true to create-pull-request action

Requires repository secrets:

  • GPG_PRIVATE_KEY - Private GPG key for signing commits
  • GPG_PASSPHRASE - Passphrase for the GPG key

Testing

Once GPG secrets are configured in the repository:

  1. Merge this PR
  2. Push any change to docs/ directory on main branch
  3. Wiki sync workflow should automatically create a signed PR
  4. All CI/CD checks should pass

Related

…gning

Added comprehensive Rust error handling documentation to CLAUDE.md:
- Never use unwrap() in production code
- Use expect() for programmer errors with descriptive messages
- Use unwrap_or()/unwrap_or_else() for safe defaults
- Use match or ? operator for proper error handling
- Include code examples and verification commands

Fixed wiki-sync workflow to support GPG commit signing:
- Added crazy-max/ghaction-import-gpg@v6 action
- Configured git_commit_gpgsign to satisfy repository rules
- Requires GPG_PRIVATE_KEY and GPG_PASSPHRASE secrets

This resolves the "Commits must have verified signatures" error
that was causing wiki sync automation to fail.
@NormB NormB merged commit 03a6ccd into main Nov 20, 2025
30 checks passed
@NormB NormB deleted the docs/rust-unwrap-best-practices branch November 20, 2025 13:53
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.

2 participants