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

docs(source-control): define when a push needs a fresh AI review round

Aperta
#6,632 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@melodic-automation-lanes ci sta già lavorando.

Dal 18/10/2026.

  • #8050 di @melodic-automation-lanes — aperta

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
50/100
Tipo di issue
Documentazione
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
shell
Ambito
documentation

Direzione di ricerca

Start with plugins/source-control/skills/pull-request and read the B2 rule, then inspect the PR monitor and the draft-PR guidance in AGENTS.md. Define the merge-commit diff test and distinguish clean merges or allowed-only file changes from code conflict resolutions; done when the skill states the rule and the monitor applies it.

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

Descrizione

needs-human needs-triage

The B2 rule in plugins/source-control/skills/pull-request treats an AI review lane as absent when it is green but posted nothing for the current head. On 2026-10-07 every merge-main push and every push that only added changelog fragments waited about 5 minutes for the lanes to re-run on code they had already reviewed. ci-status is the only required check (AGENTS.md, "Open a pull request as a draft").

An interim rule, "skip the lane wait on merge-only pushes", let #6516 merge after a merge push that had resolved a real conflict in lib/docs-cache.sh. A separate review found the resolution clean, but the auto-mode classifier flagged it as "Merge Without Review".

Proposal: state the rule in the skill.

  • A push needs a fresh review round when it changes reviewed content. Any conflict resolution in code counts as changing reviewed content.
  • A merge of main needs no fresh round when it merged cleanly, or when the only files it changed were CHANGELOG files, plugin.json versions or .changes fragments.
  • Give the test: the diff of the merge commit against its main parent, limited to non-changelog paths, is empty.

Acceptance: the skill gives the test, and the PR monitor applies it.

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

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.