fix(ci): compare check-plugins' stale-version step against the merge base
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
Línea de trabajo
Encuentra el STALE VERSION del paso del flujo de trabajo de CI llamado check-plugins e inspecciona cómo determina los cambios respecto a origin/main. Empieza leyendo ese paso y sus pruebas existentes. Añade cobertura para ambos casos de aceptación: una rama atrasada respecto a main y sin cambios en archivos compartidos pasa, mientras que una rama que cambia un archivo compartido sin un fragmento falla.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
check-plugins' STALE VERSION step fails a PR that is only behind main. It counts main's newer shared-file syncs (lib/prerequisites.mjs, lib/session-bridge/* and their plugin copies) as the PR's own changes. Examples: #8467 changed 4 files, none of them shared, and was 27 commits behind main; it failed with dozens of "changed vs origin/main but , in fragment mode, has no fragment" lines (run 39299393562). #8497 and #8489 failed the same way. Updating each PR from main clears the failure, but every update reruns full CI and both AI review lanes, which spends hosted-job concurrency and delays the merge.
Fix
Compare against the merge base (origin/main...HEAD, three dots), not the tip of main, wherever the step decides what "changed vs origin/main".
Acceptance
- A test where a branch behind main that changes no shared file passes the step, when main has since changed a shared file and its copies.
- A test that a branch which itself changes a shared file with no fragment still fails.
🤖 Generated with Claude Code
- Lenguaje dominante
- Shell
- Estrellas
- 22
- Forks
- 2
- Merge medio
- 8 h 44 min
- PR fusionados (30 d)
- 926
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de melodic-software/claude-code-plugins
-
status: ready
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
melodic-software/claude-code-plugins#8537 ·
Los mantenedores suelen responder en 1 día
-
priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
melodic-software/claude-code-plugins#8476 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
area: security needs-human priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
melodic-software/claude-code-plugins#8472 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
pr-pipeline: show a thread's review commit and outdated state in address-feedback list outputAbiertopriority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
melodic-software/claude-code-plugins#8434 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
needs-human
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
melodic-software/claude-code-plugins#8405 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de melodic-software/claude-code-plugins
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
vladmandic/sdnext#5161 ·
Los mantenedores suelen responder en 1 día
-
Missing icon: tangramAbierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
-
fix(backup-uploader): write current-round marker outside / so the container can run as non-rootAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
hiero-ledger/solo-containers#254 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
CachyOS/cachyos-hooks#23 ·
-
Remove the irispy-lmsal folderAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100