Faire échouer la CI sur les codes d’import Godot non reconnus #43

Closed
opened 2026-06-20 11:11:18 +00:00 by Reliodas · 1 comment
Owner

Le correctif #30 capture bien le code retour de , mais tout code non nul est ensuite toléré dès que existe et qu’aucun motif textuel ne correspond. Le commentaire affirme au contraire que tout code non couvert doit échouer. La logique a été reproduite avec un code 42 : le snippet sort avec succès.

Critères d’acceptation :

  • un code retour nul valide l’import si les autres contrôles sont satisfaits ;
  • un code non nul échoue par défaut ;
  • chaque éventuelle exception bénigne est reconnue par une signature précise, documentée et testable ;
  • un cache déjà présent ne suffit jamais à masquer un nouvel échec ;
  • les diagnostics fatals connus continuent à faire échouer même avec code nul ;
  • un test shell ou une validation équivalente couvre au moins code 0, code non nul inconnu et éventuel cas bénin autorisé.
Le correctif #30 capture bien le code retour de , mais tout code non nul est ensuite toléré dès que existe et qu’aucun motif textuel ne correspond. Le commentaire affirme au contraire que tout code non couvert doit échouer. La logique a été reproduite avec un code 42 : le snippet sort avec succès. Critères d’acceptation : - un code retour nul valide l’import si les autres contrôles sont satisfaits ; - un code non nul échoue par défaut ; - chaque éventuelle exception bénigne est reconnue par une signature précise, documentée et testable ; - un cache déjà présent ne suffit jamais à masquer un nouvel échec ; - les diagnostics fatals connus continuent à faire échouer même avec code nul ; - un test shell ou une validation équivalente couvre au moins code 0, code non nul inconnu et éventuel cas bénin autorisé.
Reliodas added this to the 0.15.0 milestone 2026-06-20 11:11:18 +00:00
Author
Owner

Complément technique : les commandes et chemins supprimés de la description sont godot --headless --import et .godot. Le défaut précis est que import_rc != 0 affiche seulement une note aux lignes 97–100 de .woodpecker.yml, sans exit 1.

Complément technique : les commandes et chemins supprimés de la description sont `godot --headless --import` et `.godot`. Le défaut précis est que `import_rc != 0` affiche seulement une note aux lignes 97–100 de `.woodpecker.yml`, sans `exit 1`.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: jeux/beaulieu-sur-brume#43
No description provided.