Phase 1 quick wins: bounded SHA fetches, concurrency-group fallback, paths-ignore in templates/ci.yml

Abierto
  • ¿Algo no está bien?
#147 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
76/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Tranquilo
Stack tecnológico
github-actions, shell
Área
ci-cd, devops

Línea de trabajo

Comienza con scripts/detect-changed-lectures.sh:26-27 y templates/ci.yml:24-26, y después inspecciona los desencadenadores del workflow de la plantilla y sus dos consumidores. Añade obtenciones acotadas de SHA, el fallback de concurrencia pull-request-number/run-id y un filtro fail-open paths-ignore para que los pull requests que solo afectan a la documentación omitan las previews sin interrumpir las ejecuciones manuales. Se considera terminado cuando los tres elementos de la checklist estén reflejados en la plantilla reutilizable y en el script.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

enhancement

The "one small actions-repo PR" bundle from #92 Phase 1 — three independent quick wins, all in this repo:

  • --depth=1 on the two still-unbounded SHA fetches in scripts/detect-changed-lectures.sh:26-27. Shippable today; lecture-python.myst has no copy of the script (it calls preview-netlify@v0, which invokes ours), so only the checkout side is consumer work — tracked with the consumer items.
  • Concurrency cancellation fallback in templates/ci.yml:24-26: group: ci-${{ github.event.pull_request.number || github.run_id }}. The template has the group and cancel-in-progress: true but not the fallback, and the fallback is load-bearing: both consumers also trigger on workflow_dispatch, so a verbatim copy puts every manual run in the empty group ci-, cancelling each other. Also closes a stale-overwrite hazard.
  • Paths filters (paths-ignore fail-open variant) in templates/ci.yml so docs-only PRs skip the preview — the template has no paths key at all.

Related, deliberately not here: the CLI-install gate on pull_request for both preview actions folds into #105 (same two steps it wants pinned — one PR, not two); the zstd/tag-input item stays separate because :zstd must be validated before :latest moves.

Lenguaje dominante
Shell
Estrellas
0
Forks
1
Merge medio
20 h 3 min
PR fusionados (30 d)
48

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de QuantEcon/actions

Todos los issues de QuantEcon/actions

Issues similares

Más issues de Shell/Bash

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.