EOLE

{{toc}}

h1. Tu balises quand tu commit ?

h2. Objectif

L’idéal est de faire des commits :

* Souvent

* Par type de modification

On peut ainsi mettre de côter des commits qui ne font que corriger des typos dans du code qui n’est pas le nôtre, l’idée est :

* Corriger tout ce que l’on voit, si personne d’autre ne l’a déjà fait, alors ça sera fait

* Ne pas se donner du travail supplémentaire à gérer des conflits pour des corrections typographiques et/ou mineures.

h2. N’ayons peur de rien, balisons !

h3. Les commits de notre code

Différents types de commit :

* Le commit qui introduisent de nouvelles choses (N: New) : quand on est super fort, on ne fait que ces commits là, mais quand on est super fort, on a pas besoin de git ;-) ;

* Le commit qui corrige un problème fonctionnel dans le nouveau code introduit par le type ci-dessus (F: Fix) : et oui, ça arrive d’oublier des ", ) ou ] ;

* Le commit qui corrige un problème cosmétique (C: Cosmetic) de notre travail, décomposé en 2 sous catégories :

** La correction des messages visibles par l’utilisateur (C: print:) ;

** La correction des commentaires (C: #:).

Voici un exemple type de messages de log court pour une nouvelle fonctionnalitée :

<pre>

  moi@work:~/src/bidule(moi/voyant-qui-clignote)$ git log --oneline master..HEAD

  ed002ff N: Ajout de la bibilothèque de gestion du voyant qui clignote

  7610ac6 F: Oublie d’une ) lors de l’appel à ->new()

  c44649d C: #: Mauvaise tournure de phrase: le voyant qui clignote est obligatoire

  2ed9b43 C: print: Typo: 3e personne du pluriel == ent

</pre>

Le balisage de chaque commit permet de facilement s’y retrouver lors de la réorganisation des commits pour [[GitBonnesPratiques#Publication-du-développement|publication]] :

* Les commit № 7610ac6, c44649d et 2ed9b43 doivent être fusionnées avec le commit № ed002ff et leur message de commit suprrimé

h3. Les commits du code des autres

Il est possible, et vivement recommandé par moi-même, d’introduire une autre série de balises pour tout ce qui est correction en dehors de notre travail en cours.

C’est à dire, la correction des problèmes qui étaient déjà présents avant de faire la nouvelle branche.

* Le commit qui corrige un problème fonctionnel (Fo: Fix other) ;

* Le commit qui corrige un problème cosmétique (Co: Cosmetic other), décomposé en 2 sous catégories :

** La correction des messages visibles par l’utilisateur (Co: print:) ;

** La correction des commentaires (Co: #:).

Il est ainsi aisé de savoir quels commits il faut fusionner et quels commits restent autonommes :

<pre>

  moi@work:~/src/bidule(moi/voyant-qui-clignote)$ git log --oneline master..HEAD

  ed002ff N: Ajout de la bibilothèque de gestion du voyant qui clignote

  7610ac6 F: Oublie d’une ) lors de l’appel à ->new()

  74fe39b Co: #: Erreur de description de la variable "remote" de la méthode Bidule::Truc::Machiner()

  c44649d C: #: Mauvaise tournure de phrase: le voyant qui clignote est obligatoire

  9972b58 Fo: La méthode Bidule::Truc::Biduler() n’étant jamais utilisée, personne n’a vu qu’elle ne fonctionnait pas

  9a6c6c9 Co: print: Typo: Correction du message de l’exception Bidule::Exception::NoBattery

  2ed9b43 C: print: Typo: 3e personne du pluriel == ent

</pre>

h2. Comment s’en servir ?

Lors d’un @git rebase -i HEAD~7@, l’éditeur est lancé avec le contenu suivant :

<pre>

  moi@work:~/src/bidule(moi/voyant-qui-clignote)$ git rebase -i HEAD~7

  pick ed002ff N: Ajout de la bibilothèque de gestion du voyant qui clignote

  pick 7610ac6 F: Oublie d’une ) lors de l’appel à ->new()

  pick 74fe39b Co: #: Erreur de description de la variable "remote" de la méthode Bidule::Truc::Machiner()

  pick c44649d C: #: Mauvaise tournure de phrase: le voyant qui clignote est obligatoire

  pick 9972b58 Fo: La méthode Bidule::Truc::Biduler() n’étant jamais utilisée, personne n’a vu qu’elle ne fonctionnait pas

  pick 9a6c6c9 Co: print: Typo: Correction du message de l’exception Bidule::Exception::NoBattery

  pick 2ed9b43 C: print: Typo: 3e personne du pluriel == ent

  # Rebase ed002ff..2ed9b43 onto ed002ff

  #

  # Commands:

  #  p, pick = use commit

  #  r, reword = use commit, but edit the commit message

  #  e, edit = use commit, but stop for amending

  #  s, squash = use commit, but meld into previous commit

  #  f, fixup = like "squash", but discard this commit's log message

  #  x, exec = run command (the rest of the line) using shell

  #

  # If you remove a line here THAT COMMIT WILL BE LOST.

  # However, if you remove everything, the rebase will be aborted.

  #

</pre>

Il est ainsi possible de *fusionner* les 4 commits qui nous conserne en un seul en déplaçant les lignes et en leur appliquant la commande de "fixup" :

<pre>

  moi@work:~/src/bidule(moi/voyant-qui-clignote)$ git rebase -i HEAD~7

  pick ed002ff N: Ajout de la bibilothèque de gestion du voyant qui clignote

  f 7610ac6 F: Oublie d’une ) lors de l’appel à ->new()

  f c44649d C: #: Mauvaise tournure de phrase: le voyant qui clignote est obligatoire

  f 2ed9b43 C: print: Typo: 3e personne du pluriel == ent

  pick 74fe39b Co: #: Erreur de description de la variable "remote" de la méthode Bidule::Truc::Machiner()

  pick 9972b58 Fo: La méthode Bidule::Truc::Biduler() n’étant jamais utilisée, personne n’a vu qu’elle ne fonctionnait pas

  pick 9a6c6c9 Co: print: Typo: Correction du message de l’exception Bidule::Exception::NoBattery

  # Rebase ed002ff..2ed9b43 onto ed002ff

  #

  # Commands:

  #  p, pick = use commit

  #  r, reword = use commit, but edit the commit message

  #  e, edit = use commit, but stop for amending

  #  s, squash = use commit, but meld into previous commit

  #  f, fixup = like "squash", but discard this commit's log message

  #  x, exec = run command (the rest of the line) using shell

  #

  # If you remove a line here THAT COMMIT WILL BE LOST.

  # However, if you remove everything, the rebase will be aborted.

  #

</pre>

Une fois la mise au propre de notre historique faite, nous souhaitons publier ces modifications.

Nous allons donc rebaser notre travail sur la branche @origin/master@ :

# On commence par télécharger les éventuels modifications :

<pre>

  moi@work:~/src/bidule(moi/voyant-qui-clignote)$ git fetch origin

  remote: Counting objects: 24, done.

  remote: Compressing objects: 100% (15/15), done.

  remote: Total 15 (delta 12), reused 0 (delta 0)

  Unpacking objects: 100% (15/15), done.

  From git://far-far.away.example.net/bidule.git

     7ced305..9171f31  master     -> origin/master

</pre>

# On rebase en modifiant les messages de commit afin de supprimer le balisage, il n’y a rien à fusionner :

<pre>

  moi@work:~/src/bidule(moi/voyant-qui-clignote)$ git rebase origin/master

  r ed002ff N: Ajout de la bibilothèque de gestion du voyant qui clignote

  r 74fe39b Co: #: Erreur de description de la variable "remote" de la méthode Bidule::Truc::Machiner()

  r 9972b58 Fo: La méthode Bidule::Truc::Biduler() n’étant jamais utilisée, personne n’a vu qu’elle ne fonctionnait pas

  r 9a6c6c9 Co: print: Typo: Correction du message de l’exception Bidule::Exception::NoBattery

  # Rebase ed002ff..9a6c6c9 onto 9171f31

  #

  # Commands:

  #  p, pick = use commit

  #  r, reword = use commit, but edit the commit message

  #  e, edit = use commit, but stop for amending

  #  s, squash = use commit, but meld into previous commit

  #  f, fixup = like "squash", but discard this commit's log message

  #  x, exec = run command (the rest of the line) using shell

  #

  # If you remove a line here THAT COMMIT WILL BE LOST.

  # However, if you remove everything, the rebase will be aborted.

  #

</pre>

Si un conflit est détecté sur le commit *9a6c6c9 Co: print: Typo: Correction du message de l’exception Bidule::Exception::NoBattery*,

il suffira d’annuler le rebase avec @git rebase --abort@ et de le recommencer, cette fois ci en supprimant la ligne correspondante.