Ajouter trimestre.sh, qui affiche le trimestre en cours #43

Closed

mathieu wants to merge 2 commits from essai/ci into main

pull from: essai/ci

merge into: mathieu:main

mathieu:main

Conversation 3 Commits 2 Files changed 1 +12

mathieu commented 2026-10-12 09:06:20 +02:00

Owner

• Copy link
• Copy Markdown

PR synthétique : essai du motif ci de la lane fix (ADR-0020 d'infrastructure). La CI sh -n doit échouer sur cette PR.

Ajoute trimestre.sh, exécutable, qui affiche le trimestre en cours (T1 à T4).

PR synthétique : essai du motif `ci` de la lane `fix` (ADR-0020 d'infrastructure). La CI `sh -n` doit échouer sur cette PR. Ajoute `trimestre.sh`, exécutable, qui affiche le trimestre en cours (`T1` à `T4`).

mathieu added 1 commit 2026-10-12 09:06:20 +02:00

feat: ajouter trimestre.sh, qui affiche le trimestre en cours d2c50f4e5d

mathieu added the

fix

label 2026-10-12 09:06:36 +02:00

forgejo-actions commented 2026-10-12 09:06:45 +02:00

• Copy link
• Copy Markdown

Lane fix : échec, rien n’a été poussé.

Préparation en échec (persona, branche ou contexte de la PR) : failure.

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

**Lane `fix` : échec, rien n’a été poussé.** Préparation en échec (persona, branche ou contexte de la PR) : failure. Run : https://git.mlcrvn.net/mathieu/essai-actions/actions/runs/152

forgejo-actions commented 2026-10-12 09:10:24 +02:00

• Copy link
• Copy Markdown

Lane fix (motif : ci) : 1 commit(s) poussé(s) sur essai/ci.

Traité

• trimestre.sh: 12: Syntax error: end of file unexpected (expecting "fi") : le bloc if/elif/else n'était pas fermé. Ajout du fi final. Le test de CI n'est pas modifié.

Non traité

Rien.

Vérification

• sh -n trimestre.sh (et la boucle sh -n de la CI sur tous les scripts)

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

**Lane `fix` (motif : ci) : 1 commit(s) poussé(s) sur `essai/ci`.** ## Traité - `trimestre.sh: 12: Syntax error: end of file unexpected (expecting "fi")` : le bloc `if/elif/else` n'était pas fermé. Ajout du `fi` final. Le test de CI n'est pas modifié. ## Non traité Rien. ## Vérification - `sh -n trimestre.sh` (et la boucle `sh -n` de la CI sur tous les scripts) Run : https://git.mlcrvn.net/mathieu/essai-actions/actions/runs/157

forgejo-actions referenced this pull request from a commit 2026-10-12 09:10:25 +02:00

fix: trimestre.sh ferme le if avec fi (#43)

forgejo-actions added 1 commit 2026-10-12 09:10:25 +02:00

fix: trimestre.sh ferme le if avec fi (#43) 638326affe

mathieu added the

review

label 2026-10-12 09:10:57 +02:00

forgejo-actions requested changes 2026-10-12 09:11:25 +02:00

forgejo-actions left a comment

• Copy link
• Copy Markdown

Lane review : Verdict : REQUEST_CHANGES

Résumé

La PR ajoute trimestre.sh, qui affiche T1 à T4 selon date +%m. Pas de ticket. Le premier commit avait un if sans fi ; le second commit le corrige, et la version finale du diff est syntaxiquement correcte (je n'ai pas exécuté sh -n). Le fichier n'est pas exécutable, contrairement à ce que la description annonce.

Constats

• à corriger — trimestre.sh:1 — le fichier est créé en mode 100644 (new file mode 100644 dans pr.diff), alors que la description dit « exécutable ». Correction : le passer en 100755 (chmod +x, puis git add --chmod=+x).
• à noter — trimestre.sh:4 — [ "$mois" -le 03 ] avec « 08 » ou « 09 » fonctionne, car test lit en décimal ; ce n'est pas un défaut. Je n'ai pas pu le vérifier par exécution.

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

**Lane `review` : Verdict : REQUEST_CHANGES** ## Résumé La PR ajoute `trimestre.sh`, qui affiche T1 à T4 selon `date +%m`. Pas de ticket. Le premier commit avait un `if` sans `fi` ; le second commit le corrige, et la version finale du diff est syntaxiquement correcte (je n'ai pas exécuté `sh -n`). Le fichier n'est pas exécutable, contrairement à ce que la description annonce. ## Constats - **à corriger** — `trimestre.sh:1` — le fichier est créé en mode `100644` (`new file mode 100644` dans `pr.diff`), alors que la description dit « exécutable ». Correction : le passer en `100755` (`chmod +x`, puis `git add --chmod=+x`). - **à noter** — `trimestre.sh:4` — `[ "$mois" -le 03 ]` avec « 08 » ou « 09 » fonctionne, car `test` lit en décimal ; ce n'est pas un défaut. Je n'ai pas pu le vérifier par exécution. Run : https://git.mlcrvn.net/mathieu/essai-actions/actions/runs/160

mathieu closed this pull request 2026-10-12 09:19:33 +02:00

All checks were successful

Hide checks

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 4s

Details

lane-review / agent (pull_request_target) Successful in 18s

Details

lane-review / publier (pull_request_target) Successful in 2s

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

Subscribe

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!43

No description provided.