improve documentation
Fix multiple typos in the internal documentation
Improve visual aspect and redaction
Merge request reports
Activity
- Resolved by Hugo Trentesaux
(je spamme les commits parce que j'ai besoin de savoir comment ça rend sur GitLab)
added 2 commits
added C-doc label
added RN-silent label
assigned to @HugoTrentesaux
added 10 commits
-
cd5645d2...386e4d84 - 6 commits from branch
master
- aac85da2 - doc(all): fix typos
- 65f5ac6a - doc(all): improve docs
- 79de9c30 - doc(all): update docker tag
- 1f3a0d47 - doc(end2end): detail test users
Toggle commit list-
cd5645d2...386e4d84 - 6 commits from branch
requested review from @librelois
- Resolved by Hugo Trentesaux
- Resolved by Hugo Trentesaux
@HugoTrentesaux I have 2 blocking comments.
Quand on fait des petits fix comme ça est-ce qu'il y a besoin de faire un commit séparé ou est-ce que de toute façon on squash tout ?
Est-ce qu'on garde les mêmes conventions pour le message de squash ? J'ai vu que maintenant tu utilisais des noms comme
Smith members can't revoke idty nor change its address (!102)
Remove genesis params `*certs_expire_on` (!103)
Edited by Hugo TrentesauxQuand on fait des petits fix comme ça est-ce qu'il y a besoin de faire un commit séparé ou est-ce que de toute façon on squash tout ?
Tout les commit snot squash en 1, mais c'est quand même mieux de faire un commit séparé car ça aide les reviewer à ne vérifier que les nouvelles diff, sinon on doit tout relire à chaque fois pour être sur et c'est trop long. À terme les
push --force
seront interdit (soit on rebasera via un bot soit on mergera master à la place).Est-ce qu'on garde les mêmes conventions pour le message de squash ?
Le message de squash dépend du titre de la MR, les nom des commit n'ont pas d'influence. Donc dans la pratique maintenant les nom des commit on s'en fou, on pourrait alléger/supprimer les conventions la dessus.
Le message de squash doit juste être suffisamment précis pour qu'on comprenne de quoi il s'agit en lisant l'historique, sans être trop long.
Edited by Éloïsmentioned in commit ef73a0d0