Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typo: branch name not in sync with the diagram #125

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ebouchut
Copy link

@ebouchut ebouchut commented Nov 6, 2019

The branch names in the french version (experience, server) is not in sync with the ones in the diagrams (experiment, server).
I suggest to either use the same as in the diagram (which I did in this page for now) either update the diagrams so that they use the same branch name (experience, serveur).

Let me know your thoughts.

The branch names in the french version ('experience`, `server`) is not in sync with the ones in the diagrams (`experiment`, `server`).
I suggest to either use the same as in the diagram (which I did in this page for now) either update the diagrams so that they use the same branch name (`experience`, `serveur`).

Let me know your thoughts.
@aollier
Copy link
Member

aollier commented Dec 8, 2019

Je suis plutôt d'accord avec cette correction pour garder de la cohérence entre les diagrammes et le texte.
De plus, même dans les entreprises françaises on a tendance à tout mettre en anglais, donc c'est cohérent de garder les noms anglais.
@jnavila Es-tu d'accord ?

@aollier
Copy link
Member

aollier commented Dec 8, 2019

Et pour garder de la cohérence jusqu'au bout, vu qu'on est sur la version française, @ebouchut pouvez-vous réécrire votre message de commit en français ? Corrigez votre dernier commit (git commit --amend) et faites une poussée en forçant le remplacement de ce commit (git push --force-with-lease origin patch-1).

@jnavila
Copy link
Member

jnavila commented Dec 8, 2019

Je suis partagé. Depuis qu'on a basculé les images d'origine en svg, on pourrait théoriquement traduire aussi les figures, ce qui finirait totalement la traduction. Mais, pour l'instant, c'est toujours les images png, donc pas traduites.

La proposition actuelle a le mérite de resynchroniser texte et images. Il sera temps plus tard de les traduire.

@jnavila
Copy link
Member

jnavila commented Dec 8, 2019

À noter que git-scm.com ne gère pas les images traduites (toutes les langues pointent sur les mêmes images). Avis aux amateurs.

@aollier
Copy link
Member

aollier commented Feb 27, 2021

Bonjour @ebouchut , il y a un conflit qui empêche de fusionner cette pull request. Veuillez résoudre le problème sur votre dépôt et pousser à nouveau votre branche.
Merci

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants