Skip to content

add first version translation faq-versioning.md #91

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

Merged
merged 26 commits into from
Mar 13, 2019
Merged

add first version translation faq-versioning.md #91

merged 26 commits into from
Mar 13, 2019

Conversation

salimbenfarhat
Copy link
Contributor

No description provided.

@salimbenfarhat salimbenfarhat deleted the salimbenfarhat-branch01 branch March 7, 2019 03:01
@netlify
Copy link

netlify bot commented Mar 7, 2019

Deploy preview for fr-reactjs ready!

Built with commit 9f27401

https://deploy-preview-91--fr-reactjs.netlify.com

@salimbenfarhat
Copy link
Contributor Author

test

@salimbenfarhat salimbenfarhat reopened this Mar 7, 2019
@salimbenfarhat
Copy link
Contributor Author

comment delete ma PR pour ajouter une nouvelle plus propre svp ?

@tdd
Copy link
Collaborator

tdd commented Mar 8, 2019

@salimbenfarhat du coup on considère que ta PR est prête à la revue, ou tu es encore en train de bosser dessus ?

@salimbenfarhat
Copy link
Contributor Author

@tdd Oui c'est bon prête a la revue

@tdd tdd self-requested a review March 8, 2019 14:12
@tdd tdd self-assigned this Mar 8, 2019
@tdd tdd added Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR Reviewing Un mainteneur qualifié est en train de composer la revue de code and removed Pending Review Un mainteneur qualifié doit encore faire la revue de cette PR labels Mar 8, 2019
Copy link
Collaborator

@tdd tdd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salut @salimbenfarhat !

Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main.

Bon, ça porte évidemment les stigmates d'une première trad. Couacs de typographie, de gestion des espaces, des sauts de lignes… Avec en prime un éditeur qui semble avoir bien charcuté du Markdown.

Côté terminologie, quelques gros impairs / écarts de-ci de-là, mais beaucoup de bonnes formules aussi, notamment sur la deuxième moitié du texte. Donc moi je dis bravo ! 👏

Tout est fait sous forme de Suggestions GitHub, il te suffit à chaque suggestion que tu approuves de cliquer le bouton d'ajout au lot de modifs à committer, et sur la fin de créer le commit groupé. (ce mode de regroupement en un seul commit est à favoriser absolument, mais tu devras aller dans l'onglet Files changed pour y accéder, il n'est pas utilisable dans l’onglet Conversation)

Si certaines appellent discussion, n'hésite pas, tu réponds, on en parle, on avance ensemble 🤝

Quand tout sera bouclé côté code, je ferai une repasse visuelle finale en local avant de fusionner.

Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans master !

@tdd tdd added Reviewed and removed Reviewing Un mainteneur qualifié est en train de composer la revue de code labels Mar 12, 2019
tdd and others added 10 commits March 13, 2019 17:01
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
@tdd
Copy link
Collaborator

tdd commented Mar 13, 2019

@salimbenfarhat s'il te plaît, ne fais pas un commit par suggestion ! Comme l'indique le bilan de la revue, passe dans l'onglet Files Changed et fais un add suggestion to batch sur chaque suggestion acceptée, pour finaliser le lot après coup. Ça pourrit l'histo et mes notifs sinon 😉

tdd and others added 11 commits March 13, 2019 17:04
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Co-Authored-By: salimbenfarhat <salim.benfarhat@yahoo.com>
Copy link
Contributor Author

@salimbenfarhat salimbenfarhat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check test

@salimbenfarhat
Copy link
Contributor Author

@tdd j'ai fait un bordel dsl, j'ai zappé ton truc de Files changed je suis désolé, comment je peu réparer ca

@tdd
Copy link
Collaborator

tdd commented Mar 13, 2019

@salimbenfarhat te bile pas, ce qui est fait est fait 😉 Au final les suggestions sont traitées, je peux passer à ma revue visuelle locale finale, ton boulot est donc terminé 🎉

@tdd tdd merged commit 982debf into reactjs:master Mar 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants