Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Merge queue squash commit fails required_signatures when a wrapped PR body line starts with 'committer '

Aperta
  • Qualcosa non va?
#7,279 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
55/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
git, github, github-actions, shell
Ambito
ci-cd, devtools

Direzione di ricerca

Start by inspecting the /source-control:pull-request body tooling and the pr-contract CI check mentioned in the issue. Reproduce the wrapping at 72 columns and check whether any wrapped body line starts with a Git commit header keyword. Done means the queue no longer creates an invalid signed squash commit for this case, with a check that catches the offending line before merge.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

needs-human

The merge queue removed #7225 twice with branch_protection_failure, even though its CI passed in the queue.

What happened

  • The queue's squash commit c2b54b0, committed by GitHub, shows verification.reason: invalid. The required_signatures rule (org ruleset signing, 17989003) then refuses it.
  • GitHub builds the squash message from the PR body and wraps it at about 72 columns. One wrapped line began committer identity set in the step env.
  • In the signed payload, the gpgsig block sits right after that body line, not after the real committer header. So the signing placed the signature after the first line starting committer .
  • The previous squash commit on main, 4a453b0, is valid.
  • Rewording the body so no wrapped line starts with committer fixed it.

Who acts and what they do

PR authors and the PR body tooling (/source-control:pull-request) should keep any line of the wrapped body from starting with a git header keyword, committer above all. One option is a ci-status pr-contract check that wraps the body at 72 columns and fails on ^(tree|parent|author|committer|gpgsig|encoding|mergetag) . That check would catch it before the queue does.

Evidence: gh api repos/melodic-software/claude-code-plugins/git/commits/c2b54b03e9c0bcacbf7367a2fa923d3ee308bcf9 --jq .verification

🤖 Generated with Claude Code

Lingua principale
Shell
Stelle
22
Fork
2
Merge medio
8h 25m
PR unite (30g)
944

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di melodic-software/claude-code-plugins

Tutte le issue di melodic-software/claude-code-plugins

Issue simili

Altre issue su Shell/Bash

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.