Ajouter un script annee.sh (#45) #46
No reviewers
Labels
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
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
mathieu/essai-actions!46
Loading…
Reference in a new issue
No description provided.
Delete branch "lane/code-45"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Contexte
Le ticket demande un script
annee.sh, sur le modèle dejour.sh, qui affiche l'année courante sur quatre chiffres.Closes #45
Changement
annee.sh: nouveau script#!/bin/shqui exécutedate +%Y.README.md: ajout deannee.shà la liste des scripts.Le fichier est créé en mode 644, car je n'ai pas pu faire
chmod +x(commande non permise). Les autres scripts sont en 755. Il faut donc l'exécuter avecsh annee.sh, ou corriger le mode (chmod +x annee.sh, puisgit update-index --chmod=+x annee.sh).Vérification
sh annee.sh: non lancé, la permission a été refusée. Le script se réduit àdate +%Y.sh annee.shet vérifier que la sortie fait quatre chiffres.Lane
review: Verdict : REQUEST_CHANGESRésumé
La PR ajoute
annee.sh(date +%Y) et le référence dans le README. Le contenu répond au ticket (année sur quatre chiffres), mais le fichier est créé en mode 644 alors que les autres scripts sont en 755.Constats
annee.sh:1— le fichier est créé en mode 100644 (voir le diff :new file mode 100644), donc non exécutable (./annee.shéchoue), contrairement aux autres scripts du dépôt, que la PR dit en 755. Le ticket demande un script « commejour.sh». Correction :chmod +x annee.shpuisgit update-index --chmod=+x annee.sh, et committer le mode 755.README.md:7— l'entrée est bien placée dans l'ordre alphabétique. Le script n'a pas été exécuté par l'auteur ; je ne l'ai pas exécuté non plus (pas de shell), maisdate +%Yest correct.Run : https://git.mlcrvn.net/mathieu/essai-actions/actions/runs/172
Lane
fix(motif : revue) : abandon, rien n’a été poussé.Raison
Le seul constat (« à corriger ») demande de passer
annee.shen mode 755. Il est juste : le diff montrenew file mode 100644.Je ne peux pas le corriger avec mes permissions. Edit ne change pas les modes,
chmodn'est pas autorisé, etgit update-index --chmod=+x annee.sha été refusé (seulesgit addetgit commiten écriture sont permises).git addne change pas le mode d'un fichier déjà suivi. Je n'ai pas cherché à contourner ce refus.Rien n'a été committé.
Question pour une personne : qui applique
chmod +x annee.shpuisgit update-index --chmod=+x annee.sh, et committe le mode 755, ou bien autorise cette commande git pour le workflow ?Le constat « à noter » sur
README.md:7ne demande aucune action.Run : https://git.mlcrvn.net/mathieu/essai-actions/actions/runs/173
Pull request closed