[PATCH 01 of 14 eac] * add support for the <abstract> tag under <bioghist>

Denis Laxalde denis.laxalde at logilab.fr
Wed Jul 3 10:26:15 CEST 2019


Guillaume Vandevelde a écrit :
> # HG changeset patch
> # User Guillaume Vandevelde <gvandevelde at logilab.fr>
> # Date 1560340913 -7200
> #      Wed Jun 12 14:01:53 2019 +0200
> # Node ID 732f2fb81a9707f62c1d6bbd92d7e53727db4326
> # Parent  1ff1b25a10195b1663fbf4fc2a3aeb9d3305d0c5
> # Available At http://hg.logilab.org/review/cubes/eac
> #              hg pull http://hg.logilab.org/review/cubes/eac -r 732f2fb81a97
> * add support for the <abstract> tag under <bioghist>
> * add better testing for the <abstract> tag

Guillaume, peux-tu prêter un peu plus de soin à tes messages de commit
ainsi qu'à l'historique de la série comme suggéré par Philippe ? C'est
important car d'une part tout le monde n'a pas connaissance du contexte
des changements que tu proposes et d'autre part, même en le connaissant,
il y manque les informations sur ce qui change et le pourquoi et le
comment des changements.

Sur la forme, par exemple sur ce changeset, il ne faut pas mettre une
liste à puces et il faut que la deuxième ligne du message soit vide.

Le changeset suivant contient en fait un correctif pour *ce* changeset,
il faut les fusionner.

Enfin, étant donné que la série est assez longue, je suggère d'itérer
d'abord sur un ou deux patchs jusqu'à ce qu'ils soient acceptés et
ensuite d'envoyer les autres (éventuellement au fur et à mesure aussi).

> Differential Revision: https://phab.logilab.fr/D3367

Je vois qu'il y a eu une première revue de ce patch sur Phabricator,
est-ce que ceux qui y ont participé sont inscrits à la liste ? (Ça
serait bien qu'ils le soient). Dans tous les cas, on peut enlever ces
liens dans le message étant donné que la revue se passe ici pour ce
projet.



More information about the saem-devel mailing list