Skip to content

Conversation

@deronnax
Copy link
Collaborator

@deronnax deronnax commented Nov 9, 2018

No description provided.

@deronnax
Copy link
Collaborator Author

j'exige une revue de @christopheNan !

@deronnax
Copy link
Collaborator Author

et le Travis semble pété, @JulienPalard

@JulienPalard
Copy link
Member

et le Travis semble pété, @JulienPalard

Il dit que t'as des fautes d'orthographe:

faq/design.po:429:len faq/design.po:429:x.len faq/design.po:429:len faq/design.po:429:get faq/design.po:429:key faq/design.po:429:write faq/design.po:1115:list.sort 
Copy link
Member

@JulienPalard JulienPalard left a 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 "
Copy link
Member

Choose a reason for hiding this comment

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

dupés ?

Copy link
Collaborator Author

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

Copy link
Contributor

Choose a reason for hiding this comment

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

surpris?

Copy link
Member

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.

@deronnax
Copy link
Collaborator Author

pourquoi je dois rajouter des backquotes alors qu'il n'y en a pas dans le message d'origine ?

@deronnax
Copy link
Collaborator Author

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
Copy link
Member

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator Author

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 ?

Copy link
Member

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.

deronnax and others added 17 commits November 16, 2018 01:05
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>
@deronnax
Copy link
Collaborator Author

je pense que j'ai abordé toutes les remarques. Un p'tit coup de merge siouplè @JulienPalard ?

@JulienPalard JulienPalard merged commit 175c215 into python:3.7 Nov 21, 2018
@JulienPalard
Copy link
Member

Mergé ! @deronnax n'hésite pas à enlever le "WIP" du titre de la PR quand c'est prêt à être mergé :)

@deronnax
Copy link
Collaborator Author

Ah ouais, OK, pardon. C'est noté

@deronnax deronnax deleted the design branch November 21, 2018 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants