Script de pre-commit plus robuste #623
No reviewers
Labels
No labels
devtype -- backend
devtype -- docs
devtype -- frontend
devtype -- user interface
difficulty -- easy
difficulty -- hard
difficulty -- normal
Doing
domain -- bda
domain -- bds
domain -- cof
domain -- core
domain -- kfet
Good first issue
priority -- high
priority -- low
priority -- medium
priority -- staff-wanted
status -- development
status -- discussion
status -- need review
status -- production
status -- ready to merge
status -- todo
To Do
type -- bug
type -- hygiene
type -- improvement
type -- new feature
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: DGNum/gestioCOF#623
Loading…
Reference in a new issue
No description provided.
Delete branch "Kerl/linters"
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?
Applique les changements suggérés par shellcheck sur les deux points suivants :
SC2181 : en l'occurrence, certaines erreurs étaient mal rattrapées car le
printf "OK\n"
met la variable$?
à0
SC2086 : ça n'a pas d'effet en l'état actuel des choses mais on prévient des bugs possible si jamais un nom de fichier contient un jour un caractère de type espace, \n, tabulation, etc.
Shellcheck trigger un autre warning sur ce fichier (SC1117). Je l'ignore délibérément, ce n'est pas vraiment une source de bug et il est sur le point d'être désactivé par shellcheck.
added ~64 ~59 ~28 labels
le bleu!
merci :)
merged
mentioned in commit
fc4b852bde
Preque @mpepin !
Il prend tous les fichiers comme un seul argument :-( Je peux te laisser retirer les "" dans les calls à black & cie ? (ou une toute solution qui fonctionne que tu trouveras :-P) et merge sur master si t'es sûr ? (je dois aller faire des emplettes)