mathieu/essai-actions
Watch
1
0
Fork
You've already forked essai-actions
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Ajouter siecle.sh #57

Closed
mathieu wants to merge 2 commits from essai/ci-rouge into main
pull from: essai/ci-rouge
merge into: mathieu:main
Conversation 1 Commits 2 Files changed 1 +4
mathieu commented 2026-10-14 01:22:20 +02:00
Owner

Essai de lanes#12 : la revue doit voir une CI rouge. Ne pas fusionner.

Essai de lanes#12 : la revue doit voir une CI rouge. Ne pas fusionner.
essai : siecle.sh (CI rouge, lanes#12)
All checks were successful
ci / scripts (push) Successful in 3s
Details
essai-runner / essai (push) Successful in 7s
Details
release-essai / release (push) Successful in 6s
Details
d655197dc4
essai : siecle.sh sans fi (syntaxe invalide voulue)
Some checks failed
ci / scripts (push) Failing after 4s
Details
release-essai / release (push) Successful in 7s
Details
essai-runner / essai (push) Successful in 7s
Details
ci / scripts (pull_request) Failing after 3s
Details
lane-fix / preparer (pull_request_target) Has been skipped
Details
lane-fix / agent (pull_request_target) Has been skipped
Details
lane-fix / verifier (pull_request_target) Has been skipped
Details
lane-fix / publier (pull_request_target) Has been skipped
Details
lane-review / preparer (pull_request_target) Successful in 6s
Details
lane-review / agent (pull_request_target) Successful in 27s
Details
lane-review / publier (pull_request_target) Successful in 5s
Details
fe79711238
forgejo-actions requested changes 2026-10-14 01:23:35 +02:00
forgejo-actions left a comment

Lane review : Verdict : REQUEST_CHANGES

Résumé

La PR ajoute siecle.sh, un script censé afficher le siècle courant. Pas de ticket. Le corps de la PR dit que c'est un essai de CI rouge (lanes#12) et qu'il ne faut pas fusionner. Le script final est syntaxiquement invalide et la CI est rouge.

Constats

  • bloquant — CI rouge : le job scripts échoue (run 472, ci.yml, pull_request ; le run 468 en push échoue aussi). Le journal indique siecle.sh: 5: Syntax error: end of file unexpected (expecting "fi"), puis ::error file=siecle.sh::syntaxe invalide. Correction : rétablir le fi final.
  • bloquant — siecle.sh:4 — le dernier commit (fe79711) supprime la ligne fi, donc le if de la ligne 3 n'est jamais fermé et le script ne s'exécute pas. Le message du commit précise que c'est voulu. Correction : ajouter fi après echo XXI.
  • à corriger — siecle.sh:3-4 — même avec le fi, le script n'affiche rien pour une année ≤ 2000. Il affiche XXI pour toute année > 2000, y compris 2000 exclu à tort : l'an 2000 appartient au XXe siècle, mais 2001 et les années suivantes jusqu'à 2100 sont bien le XXIe, et à partir de 2101 le résultat devient faux (XXII). Correction : calculer le siècle à partir de l'année, ou borner la condition.
  • à noter — La PR se déclare comme un essai à ne pas fusionner. Ce constat tient donc surtout à la CI rouge.

Run : https://git.mlcrvn.net/mathieu/essai-actions/actions/runs/255

**Lane `review` : Verdict : REQUEST_CHANGES** ## Résumé La PR ajoute `siecle.sh`, un script censé afficher le siècle courant. Pas de ticket. Le corps de la PR dit que c'est un essai de CI rouge (lanes#12) et qu'il ne faut pas fusionner. Le script final est syntaxiquement invalide et la CI est rouge. ## Constats - **bloquant** — CI rouge : le job `scripts` échoue (run 472, `ci.yml`, pull_request ; le run 468 en push échoue aussi). Le journal indique `siecle.sh: 5: Syntax error: end of file unexpected (expecting "fi")`, puis `::error file=siecle.sh::syntaxe invalide`. Correction : rétablir le `fi` final. - **bloquant** — `siecle.sh:4` — le dernier commit (fe79711) supprime la ligne `fi`, donc le `if` de la ligne 3 n'est jamais fermé et le script ne s'exécute pas. Le message du commit précise que c'est voulu. Correction : ajouter `fi` après `echo XXI`. - **à corriger** — `siecle.sh:3-4` — même avec le `fi`, le script n'affiche rien pour une année ≤ 2000. Il affiche `XXI` pour toute année > 2000, y compris 2000 exclu à tort : l'an 2000 appartient au XXe siècle, mais 2001 et les années suivantes jusqu'à 2100 sont bien le XXIe, et à partir de 2101 le résultat devient faux (XXII). Correction : calculer le siècle à partir de l'année, ou borner la condition. - **à noter** — La PR se déclare comme un essai à ne pas fusionner. Ce constat tient donc surtout à la CI rouge. Run : https://git.mlcrvn.net/mathieu/essai-actions/actions/runs/255
mathieu closed this pull request 2026-10-14 02:39:56 +02:00
Some checks failed
ci / scripts (push) Failing after 4s
Details
release-essai / release (push) Successful in 7s
Details
essai-runner / essai (push) Successful in 7s
Details
ci / scripts (pull_request) Failing after 3s
Details
lane-fix / preparer (pull_request_target) Has been skipped
Details
lane-fix / agent (pull_request_target) Has been skipped
Details
lane-fix / verifier (pull_request_target) Has been skipped
Details
lane-fix / publier (pull_request_target) Has been skipped
Details
lane-review / preparer (pull_request_target) Successful in 6s
Details
lane-review / agent (pull_request_target) Successful in 27s
Details
lane-review / publier (pull_request_target) Successful in 5s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Compat/Breaking
Breaking change that won't be backward compatible

  
Décommissions et revue d'usage

   
Garde-fous

   
Harnais d'évaluation

   
Kind/Bug
Something is not working

  
Kind/Documentation
Documentation changes

  
Kind/Enhancement
Improve existing functionality

  
Kind/Feature
New functionality

  
Kind/Security
This is security issue

  
Kind/Testing
Issue or pull request related to testing

  
Outillage agent
  
Priority
Critical
 The priority is critical

  
Priority
High
 The priority is high

  
Priority
Low
 The priority is low

  
Priority
Medium
 The priority is medium

  
Reviewed
Confirmed
 Issue has been confirmed

  
Reviewed
Duplicate
 This issue or pull request already exists

  
Reviewed
Invalid
 Invalid issue

  
Reviewed
Won't Fix
 This issue won't be fixed

  
Sauvegarde hors site

   
Skills
  
Status
Abandoned
 Somebody has started to work on this but abandoned work

  
Status
Blocked
 Something is blocking this issue or pull request

  
Status
Need More Info
 Feedback is required to reproduce issue or to continue work

  
code
Lane code (ADR-0020) : implémente le ticket, pousse une branche, ouvre une PR

  
fix
Déclenche la lane fix (ADR-0020) : corrige la PR d'après sa CI en échec ou ses revues, pousse sur sa branche

  
needs-info
Tri : il manque une information que seul l'auteur peut donner

  
ready-for-agent
Tri : plus aucune décision à prendre, implémentable par un agent (ne déclenche rien)

  
ready-for-human

   
review
Déclenche la lane review (ADR-0020) : relit la PR et publie un verdict

  
triage
Déclenche la lane triage (ADR-0020) : trie le ticket et propose des étiquettes

No labels
Compat/Breaking
Décommissions et revue d'usage
Garde-fous
Harnais d'évaluation
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Outillage agent
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Sauvegarde hors site
Skills
Status
Abandoned
Status
Blocked
Status
Need More Info
code
fix
needs-info
ready-for-agent
ready-for-human
review
triage
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
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
mathieu/essai-actions!57
No description provided.