-
- Notifications
You must be signed in to change notification settings - Fork 264
WIP #404
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
WIP #404
Conversation
| j'exige une revue de @christopheNan ! |
| et le Travis semble pété, @JulienPalard |
Il dit que t'as des fautes d'orthographe: |
JulienPalard left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quelques bugs rst et surtout de la typographie :)
faq/design.po Outdated
| "Dans les situations où la performance est importante, faire une copie de la " | ||
| "liste juste pour la trier serait un gaspillage. Par conséquent, :meth:`list." | ||
| "sort` trie la liste en place. Afin de vous le rappeler, il ne retourne pas " | ||
| "la liste triée. De cette façon, vous ne serez pas dupé en écrasant " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dupés ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oui, c'est la traduction de "fooled" que donnent deepl, google et bing. J'ai cherché autre chose, pas trouvé de meilleur terme (mais je reconnais qu'il sonne pas très naturel). Si tu as mieux qui te vient à l'esprit, je prends
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
surpris?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Non non je ne suis pas surpris, je proposais juste de mettre un s à dupé. vous ne serez pas dupés.
| pourquoi je dois rajouter des backquotes alors qu'il n'y en a pas dans le message d'origine ? |
| j'ai abordé toutes tes remarques sauf "fooled/dupés" parce que j'ai pas mieux |
| git checkout -- . | ||
| pomerge --to-files **/*.po | ||
| poindent --modified | ||
| powrap --modified |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
d'où sort ce changement ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haha j'me souvenait même pas l'avoir fait, j'ai cru que c'était toi qui l'avait fait þ
A mon avis t'as du rebase un merge en essayant de te mettre à jour sur la master, regardes là y'a des commits qui sont pas à toi : https://github.com/python/python-docs-fr/pull/404/commits
Ça se nettoie facilement, c'est git :) un petit git rebase -i a03190814fe3582a049cc70c8d7f5cfc7150e598 et tu vire les lignes qui ne t'appartiennet pas et c'est réglé. Ça t'imposera évidement de "push force" vu que supprimer des commits dans l'historique c'est changer l'historique mais c'est pas gênant un push force sur une branche sur laquelle tu est tout seul sur ton fork.
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
Co-Authored-By: deronnax <deronnax@users.noreply.github.com>
| je pense que j'ai abordé toutes les remarques. Un p'tit coup de merge siouplè @JulienPalard ? |
| Mergé ! @deronnax n'hésite pas à enlever le "WIP" du titre de la PR quand c'est prêt à être mergé :) |
| Ah ouais, OK, pardon. C'est noté |
No description provided.