full-day workshop: Correct the CI workflow triggers and first-run verification

Aperta
#264 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
76/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
github-actions
Ambito
ci-cd, documentation

Direzione di ricerca

Inizia da content/full-day/4-testing.md e confronta le relative spiegazioni dei trigger con .github/workflows/server-test.yml. Verifica il flusso della prima esecuzione in un repository template vuoto, incluse le modifiche al workflow, le modifiche al server, le pull request e l'esecuzione su richiesta. Il lavoro è completato quando gli eventi documentati e i passaggi di verifica corrispondono alle esecuzioni osservate di GitHub Actions.

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

Descrizione

priority: P0

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

  • merge is removed as an event name and explained as a push to the target branch.
  • The example includes workflow_dispatch when 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.

Lingua principale
Python
Stelle
80
Fork
161
Merge medio
31m
PR unite (30g)
1

Guida per i contributori

Apri la guida per i contributori

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 github-samples/pets-workshop

Tutte le issue di github-samples/pets-workshop

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.