Skip to content

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

Merged
merged 20 commits into from
May 23, 2020
Merged

modification fichier idle #1175

merged 20 commits into from
May 23, 2020

Conversation

fsetton
Copy link
Contributor

@fsetton fsetton commented Mar 3, 2020

Closes #1168

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.

Merci pour cet trad ! J'ai trouvé qq petits soucis.

@deronnax deronnax self-assigned this May 3, 2020
@deronnax deronnax requested review from christopheNan and awecx May 4, 2020 16:24
@deronnax
Copy link
Collaborator

deronnax commented May 4, 2020

@christopheNan @awecx que l'un de vous approuve et je merge. À priori elle est bonne.

@deronnax deronnax dismissed JulienPalard’s stale review May 4, 2020 16:25

Julien pas dispo en ce moment

@fsetton
Copy link
Contributor Author

fsetton commented May 4, 2020

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 ?

@fsetton fsetton closed this May 4, 2020
@deronnax
Copy link
Collaborator

deronnax commented May 4, 2020

pourquoi tu fermes ta PR ?

@awecx awecx reopened this May 4, 2020
@deronnax
Copy link
Collaborator

deronnax commented May 9, 2020

@fsetton tu veux continuer à travailler sur la PR ou on intègre les suggestions pour toi 😇 ?

@fsetton
Copy link
Contributor Author

fsetton commented May 9, 2020

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

@deronnax
Copy link
Collaborator

deronnax commented May 9, 2020

Hum. Quelle est l'erreur ?

@fsetton
Copy link
Contributor Author

fsetton commented May 9, 2020

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.

@deronnax
Copy link
Collaborator

deronnax commented May 9, 2020

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 ?

@python python deleted a comment from fsetton May 9, 2020
@deronnax
Copy link
Collaborator

deronnax commented May 9, 2020

(c'est pas prudent de laisser ses coordonnés sur un site web public :) )

fsetton and others added 2 commits May 10, 2020 14:20
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>
@fsetton fsetton removed their assignment May 10, 2020
@deronnax deronnax requested a review from awecx May 10, 2020 20:54
Copy link
Contributor

@christopheNan christopheNan left a 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 "
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"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 "

?

Copy link
Contributor Author

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...

Copy link
Contributor

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

J'opte pour ''ajoute''

fsetton and others added 2 commits May 13, 2020 13:04
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>
fsetton and others added 6 commits May 13, 2020 13:06
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>
@deronnax
Copy link
Collaborator

@fsetton rappelle-toi que ça peut aller plus vite avec "add suggestion to batch" dans l'onglet "files changed" en haut ;)

@deronnax deronnax requested a review from christopheNan May 21, 2020 14:41
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Copy link
Contributor Author

@fsetton fsetton left a comment

Choose a reason for hiding this comment

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

Ok

deronnax and others added 2 commits May 22, 2020 12:42
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@deronnax
Copy link
Collaborator

@awecx @christopheNan approval plz

deronnax and others added 3 commits May 22, 2020 13:36
Co-authored-by: Antoine <43954001+awecx@users.noreply.github.com>
Co-authored-by: Antoine <43954001+awecx@users.noreply.github.com>
Copy link

@awecx awecx left a comment

Choose a reason for hiding this comment

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

Validay.

@deronnax
Copy link
Collaborator

deronnax commented May 22, 2020

il faut également l'approbation de @christopheNan pour qu'on puisse la merger :(

Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

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

👍

@deronnax
Copy link
Collaborator

Et hop c'est mergé. Merci François, merci tous les autres !

@deronnax deronnax merged commit 1bf9a2f into python:3.8 May 23, 2020
@deronnax deronnax mentioned this pull request May 24, 2020
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.

Je travaille sur library/idle.po
5 participants