-
Notifications
You must be signed in to change notification settings - Fork 266
modification fichier idle #1175
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
Conversation
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.
Merci pour cet trad ! J'ai trouvé qq petits soucis.
@christopheNan @awecx que l'un de vous approuve et je merge. À priori elle est bonne. |
Si c'est fini, c'est fini. Par contre, j'ai un problème et je n'ai pas pu passer le git push pour mon fichier c-api/conversion.po que j'ai envoyé par mail à Julien. Quelqu'un peut s'en charger et me dire quand je peux fermer ma tâche ? |
pourquoi tu fermes ta PR ? |
@fsetton tu veux continuer à travailler sur la PR ou on intègre les suggestions pour toi 😇 ? |
En fait, je ne suis pas à l'aise avec le fonctionnement de github. J'ai toujours peur de faire une connerie. Il vaudrait mieux que ce soit quelqu'un d'autre qui s'en charge. Je voudrais me focaliser uniquement sur la traduction à proprement parler. J'ai même des soucis avec le git bash qui ne veut pas prendre en compte mon dernier git push sur le fichier c-api/conversion.po |
Hum. Quelle est l'erreur ? |
Je ne sais pas. J'ai l'impression que le push ne se faisait pas. J'ai transmis le fichier à Julien. Il faut voir avec lui et me dire si c'est fini pour que je ferme la tâche dans GitHub. |
Je pense que Julien ne va pas être dispo avant un très long moment avec le confinement. Tu veux qu'on tente le remote desktop voir si on peut régler ça ? |
(c'est pas prudent de laisser ses coordonnés sur un site web public :) ) |
1ère revue modification fichier idle Co-authored-by: Antoine <43954001+awecx@users.noreply.github.com>
Ca me semble un peu lourd comme formulation. De plus ''de manière spéciale'' ne me semble pas très clair. Co-authored-by: Antoine <43954001+awecx@users.noreply.github.com>
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.
Juste quelques points.
library/idle.po
Outdated
|
||
#: ../Doc/library/idle.rst:309 | ||
msgid "" | ||
"Open a column to the left of the edit window which shows the number of each " | ||
"line of text. The default is off, which may be changed in the preferences " | ||
"(see :ref:`Setting preferences <preferences>`)." | ||
msgstr "" | ||
"Ouvre une colonne à gauche de la fenêtre d'édition qui indique le numéro de " |
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.
"Ouvre une colonne à gauche de la fenêtre d'édition qui indique le numéro de " | |
"Place une colonne à gauche de la fenêtre d'édition qui indique le numéro de " |
?
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.
Peut-être "Crée" pour indiquer que la colonne n'existait pas avant...
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.
Ou "ajoute".
Sauf "Ouvre" qui est ambigu car il fait penser à une nouvelle fenêtre.
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.
J'opte pour ''ajoute''
ok Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
d'accord Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Ok Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Ok Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Ok Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Ok Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Oui c'est mieux Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Je n'osais pas rajouter des guillemets lorsqu'ils n'étaient pas présents dans le version originale. Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@fsetton rappelle-toi que ça peut aller plus vite avec "add suggestion to batch" dans l'onglet "files changed" en haut ;) |
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
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.
Ok
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@awecx @christopheNan approval plz |
Co-authored-by: Antoine <43954001+awecx@users.noreply.github.com>
Co-authored-by: Antoine <43954001+awecx@users.noreply.github.com>
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.
Validay.
il faut également l'approbation de @christopheNan pour qu'on puisse la merger :( |
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.
👍
Et hop c'est mergé. Merci François, merci tous les autres ! |
Closes #1168