Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • G gestioCOF
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 76
    • Issues 76
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 16
    • Merge requests 16
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Klub Dev ENSKlub Dev ENS
  • gestioCOF
  • Merge requests
  • !319

Script de pre-commit plus robuste

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Martin Pepin requested to merge Kerl/linters into aureplop/linters Oct 06, 2018
  • Overview 3
  • Commits 2
  • Pipelines 0
  • Changes 2

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.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: Kerl/linters