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

fix(ci): signed-commit check fails pr-refine on PRs over 250 commits even when the lane made no commits

Aperta
  • Qualcosa non va?
#7,256 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à
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
57/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
github-actions, shell
Ambito
ci-cd

Direzione di ricerca

Start with .github/actions/check-signed-commits/check-signed-commits.mjs, especially the 250-commit cap, completeness check, and escalation label request. Trace how the action identifies commits made by the lane and how it handles a lane with no commits; use the observed run on #7009 as a reference. Done means the check covers only the lane's commits, skips empty lanes, and a failed label write warns without failing the job.

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

Descrizione

needs-triage priority: medium

Problem

On a pull request with more than 250 commits, pr-refine / fix-docs fails even when the lane made no commits.

  • .github/actions/check-signed-commits/check-signed-commits.mjs:14 sets LISTED_COMMITS_CAP=250. Past the cap, line 92 sets complete=false, so allVerified is false even when every listed commit is verified.
  • The escalation then POSTs a label to issues/<n>/labels. That fails with 403 because the token has no issues write, and the failed POST becomes a hard failure.
  • The check runs even when the lane produced no commits ("The activity made no commits"), so there is nothing of the lane's to check.

Observed on #7009 (259 commits), run 38179498135: check-run summary "A commit on the pull request is not GitHub-verified."; step "Check the lane's commits are signed": "POST .../issues/7009/labels failed with status 403".

Expected

  • Skip the signed-commit check when the lane made no commits.
  • Check only the lane's own commits, not the PR's full history, so the 250-commit list cap does not apply.
  • A failed escalation label write should be a warning, not a job failure.
Lingua principale
Shell
Stelle
22
Fork
2
Merge medio
8h 27m
PR unite (30g)
957

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.