github-samples / github-samples/pets-workshop
full-day workshop: Make the generated devcontainer startup portable and non-blocking
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 80
- Forks
- 161
- Ø Merge
- 31 Min.
- Gemergte PRs (30 T.)
- 1
Beschreibung
Problem
Step 0 allows a repository name chosen by the learner or workshop leader, but content/full-day/3-codespaces.md hard-codes this startup command:
"postStartCommand": "chmod +x /workspaces/pets-workshop/app/scripts/start-app.sh && /workspaces/pets-workshop/app/scripts/start-app.sh"
It fails when the generated repository is not named pets-workshop. In addition, start-app.sh deliberately waits for both application processes and does not exit, so using it directly as a lifecycle command can leave the Codespaces startup task running indefinitely.
Suggested change
Use the dev container workspace variable or a workspace-relative command, and launch the long-running application through a non-blocking mechanism appropriate for dev containers.
Acceptance criteria
- The command works for any repository name allowed by Step 0.
- The dev container lifecycle command completes after starting the services.
- Flask and Astro logs remain accessible to the learner.
- Both services restart predictably after a Codespace stop/start.
- Service cleanup does not leave duplicate processes.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit content/full-day/3-codespaces.md und app/scripts/start-app.sh; untersuche, wie postStartCommand generiert wird und wie das Skript beide Dienste behandelt. Teste mit einem Repository-Namen, der nicht pets-workshop ist, sowie mit dem Stoppen und Starten eines Codespace. Überprüfe dabei den Abschluss des Lebenszyklus, sichtbare Flask- und Astro-Logs, vorhersehbare Neustarts und das Ausbleiben doppelter Prozesse.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- flask, python, shell
- Bereich
- developer-experience, devops
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 55/100