1-hour workshop: Fail startup clearly when dependencies or services do not start
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 55/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Domaine
- developer-experience, devops
Piste de recherche
Commencez par app/scripts/start-app.sh et app/scripts/start-app.ps1, en suivant l’installation des dépendances, l’attente fixe de cinq secondes, l’affichage des URL des services et les chemins de nettoyage. Exécutez les deux scripts de démarrage en simulant un échec de l’installation de Python ou de Node ainsi qu’un échec du démarrage de Flask ou d’Astro ; le travail est terminé lorsque les échecs renvoient un statut différent de zéro et explicite, que les processus gérés s’arrêtent ensemble, que les URL n’apparaissent qu’une fois les deux services prêts et que le nettoyage fonctionne toujours avec Ctrl+C ou après un démarrage partiel.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
The learner setup depends entirely on the startup scripts, but failure handling can report a misleading success state. In app/scripts/start-app.sh, npm install is not checked before npm run dev starts. In app/scripts/start-app.ps1, a failed Node dependency installation emits a warning and continues. Both scripts wait a fixed five seconds and then print both service URLs without checking that Flask and Astro are ready.
This makes common setup failures look like browser or workshop problems and gives independent learners little actionable feedback.
Suggested change
Make both startup scripts stop on dependency installation failures and verify service/process readiness before announcing success.
Acceptance criteria
- A failed Python or Node dependency install exits nonzero with a clear message.
- Failure of either Flask or Astro stops the other managed process.
- URLs are printed only after both services are ready.
- Cleanup still works on Ctrl+C and partial startup failures.
- macOS/Linux and Windows paths have equivalent behavior.
- Langage dominant
- Python
- Étoiles
- 80
- Forks
- 169
- Merge moyen
- 31 min
- PR mergées (30 j)
- 1
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de github-samples/pets-workshop
-
priority: P0
Difficulté 2/5 1-2 jours Accessibilité débutants 78/100
github-samples/pets-workshop#268 · 1 commentaire ·
-
priority: P0
Difficulté 1/5 1-3 heures Accessibilité débutants 88/100
github-samples/pets-workshop#267 · 1 commentaire ·
-
priority: P1
Difficulté 1/5 1-3 heures Accessibilité débutants 92/100
github-samples/pets-workshop#261 · 1 commentaire ·
-
priority: P1
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
github-samples/pets-workshop#257 · 1 commentaire ·
-
priority: deferred
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
github-samples/pets-workshop#278 · 1 commentaire ·
Toutes les issues de github-samples/pets-workshop
Issues similaires
-
Update Python support to 3.15Ouvertepython-version
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour
-
bug javascript P2-medium python release:v3.1
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
adrirubio/claude-deck#546 ·
Les mainteneurs répondent en général sous 1 jour
-
area: desktop area: website priority: P2 type: feature
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
appandflow/stim#3411 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 Moins d'une heure Accessibilité débutants 88/100
baptistehamon/lsapy#185 ·
Les mainteneurs répondent en général sous 1 jour