Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
deronnax merged 20 commits intopython:3.8fromfsetton:library-idle
May 23, 2020
Merged

modification fichier idle#1175

deronnax merged 20 commits intopython:3.8fromfsetton:library-idle
May 23, 2020

Conversation

fsetton
Copy link
Contributor

@fsettonfsetton commentedMar 3, 2020•
edited by awecx
Loading

Closes#1168

Copy link
Member

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

@deronnaxderonnax self-assigned thisMay 3, 2020
@deronnax
Copy link
Collaborator

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

@deronnaxderonnax dismissedJulienPalard’sstale reviewMay 4, 2020 16:25

Julien pas dispo en ce moment

@fsetton
Copy link
ContributorAuthor

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 ?

@fsettonfsetton closed thisMay 4, 2020
@deronnax
Copy link
Collaborator

pourquoi tu fermes ta PR ?

@awecxawecx reopened thisMay 4, 2020
@deronnax
Copy link
Collaborator

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

@fsetton
Copy link
ContributorAuthor

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

Hum. Quelle est l'erreur ?

@fsetton
Copy link
ContributorAuthor

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

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 ?

@pythonpython deleted a comment fromfsettonMay 9, 2020
@deronnax
Copy link
Collaborator

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

fsettonand others added2 commitsMay 10, 2020 14:20
1ère revue modification fichier idleCo-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>
@fsettonfsetton removed their assignmentMay 10, 2020
@deronnaxderonnax requested a review fromawecxMay 10, 2020 20:54
Copy link
Contributor

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


#: ../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
ContributorAuthor

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

deronnax reacted with thumbs up emoji
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
ContributorAuthor

Choose a reason for hiding this comment

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

J'opte pour ''ajoute''

fsettonand others added2 commitsMay 13, 2020 13:04
okCo-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
d'accordCo-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
fsettonand others added6 commitsMay 13, 2020 13:06
OkCo-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
OkCo-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
OkCo-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
OkCo-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Oui c'est mieuxCo-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 ;)

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Copy link
ContributorAuthor

@fsettonfsetton left a comment

Choose a reason for hiding this comment

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

Ok

deronnaxand others added2 commitsMay 22, 2020 12:42
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@deronnax
Copy link
Collaborator

@awecx@christopheNan approval plz

deronnaxand others added3 commitsMay 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
Contributor

@awecxawecx 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 commentedMay 22, 2020•
edited
Loading

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

Copy link
Contributor

@christopheNanchristopheNan 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 !

@deronnaxderonnax merged commit1bf9a2f intopython:3.8May 23, 2020
@deronnaxderonnax mentioned this pull requestMay 24, 2020
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@JulienPalardJulienPalardJulienPalard left review comments

@deronnaxderonnaxderonnax left review comments

@christopheNanchristopheNanchristopheNan approved these changes

@awecxawecxawecx approved these changes

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Je travaille sur library/idle.po
5 participants
@fsetton@deronnax@JulienPalard@christopheNan@awecx
[8]ページ先頭
©2009-2025 Movatter.jp