full-day workshop: Correct the CI workflow triggers and first-run verification
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
- Documentación
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- github-actions
- Área
- ci-cd, documentation
Línea de trabajo
Comienza con content/full-day/4-testing.md y compara sus explicaciones de los triggers con .github/workflows/server-test.yml. Verifica el flujo de la primera ejecución en un repositorio de plantilla nuevo, incluidos los cambios en el workflow, los cambios en el servidor, los pull requests y la ejecución bajo demanda. Se considera terminado cuando los eventos y los pasos de verificación documentados coinciden con las ejecuciones observadas de GitHub Actions.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
content/full-day/4-testing.md describes merge as a GitHub Actions event, but there is no merge workflow event; a merge to main is normally handled as a push. Learners are also told to request an on-demand trigger, while the example omits workflow_dispatch.
The example restricts both push and pull_request to app/server/**. The first commit adds only .github/workflows/server-test.yml, so the text's claim that this push triggers the workflow is false: the path filter prevents that first run. Learners can reach the Actions page with no run to inspect.
Suggested change
Teach the actual event semantics and provide an example whose triggers support every verification step in the exercise.
Acceptance criteria
-
mergeis removed as an event name and explained as a push to the target branch. - The example includes
workflow_dispatchwhen on-demand execution is required. - Adding or changing the workflow itself triggers a run, or learners are explicitly told to dispatch it manually.
- Server changes still trigger push and pull-request runs.
- The exact first-run flow is tested in a fresh template repository.
Related: #236 separately tracks the pytest/unittest inconsistency.
- Lenguaje dominante
- Python
- Estrellas
- 80
- Forks
- 161
- Merge medio
- 31 min
- PR fusionados (30 d)
- 1
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 github-samples/pets-workshop
-
priority: P0
Dificultad 2/5 1-2 días Aptitud para principiantes 78/100
github-samples/pets-workshop#268 · 1 comentario ·
-
priority: P0
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
github-samples/pets-workshop#267 · 1 comentario ·
-
priority: P1
Dificultad 1/5 1-3 horas Aptitud para principiantes 92/100
github-samples/pets-workshop#261 · 1 comentario ·
-
priority: P1
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
github-samples/pets-workshop#257 · 1 comentario ·
-
priority: deferred
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
github-samples/pets-workshop#278 · 1 comentario ·
Todos los issues de github-samples/pets-workshop
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
anthropics/skills#1811 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
speaches-ai/speaches#678 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
datalayer/mcp-compose#42 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
conda-forge/spacy-feedstock#177 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
UKGovernmentBEIS/inspect_evals#2523 ·