Skip to content
Snippets Groups Projects

docs : fix typos tuto dev durs module

Merged Ghost User requested to merge counter/fix_typo_tuto_dev_module into dev

Avant il y avait un p en trop à bibliothèque dans les phrases:

"Une bibliopthèque est utilisée exclusivement par des modules : dossier modules-common." et "Une bibliopthèque est utilisée exclusivement par un seul module : dossier modules-lib/{MODULE_NAME}."

La lettre en trop a été supprimée dans ce commit.

Edited by Éloïs

Merge request reports

Checking pipeline status.

Approval is optional

Merged by ÉloïsÉloïs 6 years ago (Jan 26, 2019 5:12pm UTC)

Merge details

  • Changes merged into dev with 4d8648d7.
  • Deleted the source branch.

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 changed the description

    changed the description

  • Merci pour ta 1ère contribution @counter-reverse :)

    Mes retours :

    1. Le titre est trop vague, il faut préciser dans le tire de le MR que cela concerne la documentation et indiquer quel tuto est concerné.
    2. Le nom du commit ne respecte pas les conventions git : https://git.duniter.org/nodes/rust/duniter-rs/blob/dev/doc/fr/conventions-git.md#nommage-des-commit (il faut que tu modifie le nom de ton commit avec un amend). De plus le nom de ton commit est trop vague, précise le tuto concerné dans le nom du commit.

    Répond moi par commentaire (en me taggant) quand tu aura pusher un correctif traitant mes retours.

    Pour tes prochaines MR : ne tag jamais personne dans la description de la MR, rajoute un commentaire pour faire cela :)

  • Ghost User added 1 commit

    added 1 commit

    Compare with previous version

  • Ghost User added 1 commit

    added 1 commit

    Compare with previous version

  • Ghost User changed title from Some typos removed. to Quelques fautes de frappes retirées dans un des fichiers de documentation à l'adresse doc/fr/developper-un-module-durs.md.

    changed title from Some typos removed. to Quelques fautes de frappes retirées dans un des fichiers de documentation à l'adresse doc/fr/developper-un-module-durs.md.

  • Ghost User changed title from Quelques fautes de frappes retirées dans un des fichiers de documentation à l'adresse doc/fr/developper-un-module-durs.md. to Quelques fautes de frappe retirées dans un des fichiers de documentation à l'adresse doc/fr/developper-un-module-durs.md.

    changed title from Quelques fautes de frappes retirées dans un des fichiers de documentation à l'adresse doc/fr/developper-un-module-durs.md. to Quelques fautes de frappe retirées dans un des fichiers de documentation à l'adresse doc/fr/developper-un-module-durs.md.

  • Ghost User changed the description

    changed the description

  • Author Contributor

    Tout semble en ordre pour faire la MR. N'est-ce pas @librelois ?

    Non, le type de commits est [docs] et pas [doc], et le nom du commit ne spécifie toujours pas le tuto concerné, or c'est ça qui vas rester donc c'est le plus important. Le titre de la MR c'est moins important.

    Edited by Éloïs
  • Éloïs changed title from Quelques fautes de frappe retirées dans un des fichiers de documentation à l'adresse doc/fr/developper-un-module-durs.md. to docs : fix typos tuto dev durs module

    changed title from Quelques fautes de frappe retirées dans un des fichiers de documentation à l'adresse doc/fr/developper-un-module-durs.md. to docs : fix typos tuto dev durs module

  • @counter-reverse le nom du commit ne spécifie toujours pas le tuto concerné, et en plus le type de commit choisir n'existe pas (c'est docs et pas doc), un nom de commit pertinent serait par exemple : [docs] dev durs module FR : fix typos

  • Ghost User added 1 commit

    added 1 commit

    • 4d8648d7 - [docs] dev durs module FR : fix typos

    Compare with previous version

  • Author Contributor

    Je viens de le modifier.

  • @librelois review ok

    Merci pour ta 1ère contribution @counter-reverse :)

  • merged

Please register or sign in to reply
Loading