Skip to content
Snippets Groups Projects

improve documentation

Merged Hugo Trentesaux requested to merge hugo-fix-typo into master
All threads resolved!

Fix multiple typos in the internal documentation

Improve visual aspect and redaction

Edited by Hugo Trentesaux

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Éloïs
  • @HugoTrentesaux I have 2 blocking comments.

  • Hugo Trentesaux added 6 commits

    added 6 commits

    Compare with previous version

  • 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 Trentesaux
  • Hugo Trentesaux resolved all threads

    resolved all threads

  • 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 ?

    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ïs
  • Éloïs approved this merge request

    approved this merge request

  • Hugo Trentesaux mentioned in commit ef73a0d0

    mentioned in commit ef73a0d0

  • merged

  • Please register or sign in to reply
    Loading