Skip to content

Conversation

@Seluj78
Copy link
Collaborator

@Seluj78 Seluj78 commented Nov 15, 2018

No description provided.

Co-Authored-By: Seluj78 <jlasne@student.42.fr>
Co-Authored-By: Seluj78 <jlasne@student.42.fr>
@Seluj78
Copy link
Collaborator Author

Seluj78 commented Dec 18, 2018

Le force push c'est un rebase 👍

@Seluj78
Copy link
Collaborator Author

Seluj78 commented Dec 18, 2018

@JulienPalard @deronnax Grosse relecture done 👌 Vous pouvez repasser si le coeur vous en dit. La fin doit etre moins bien relue que le debut FYI

Co-Authored-By: Seluj78 <jlasne@student.42.fr>
@Seluj78
Copy link
Collaborator Author

Seluj78 commented Dec 19, 2018

@JulienPalard Tu peux relancer le build travis pour cette PR ? Je n'arrive pas a voir ou est l'erreur et je me demande si ce n'est pas buggé la :/

@JulienPalard
Copy link
Member

@Seluj78 non il y a bien une erreur sur le msgid indiqué par pospell.

"être utilisée à cette fin. En général, la majorité de l'application est en"
"le code natif, et une partie invoquera soit python.exe ou utilisera "

le en a la fin de la première ligne est collé au le en début de 2eme ligne, ça crée le mot "enle" qui n'existe pas.

@Seluj78
Copy link
Collaborator Author

Seluj78 commented Dec 20, 2018

Je suis aveugle... Merci @JulienPalard

Co-Authored-By: Seluj78 <jlasne@student.42.fr>
using/windows.po Outdated
#: ../Doc/using/windows.rst:988
msgid "Embedding Python"
msgstr ""
msgstr "Incorporation de Python"
Copy link
Member

Choose a reason for hiding this comment

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

Dans le reste de la doc on a traduit par intégré : https://docs.python.org/fr/3/extending/embedding.html

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
msgstr "Incorporation de Python"
msgstr "Intégration de Python"

Co-Authored-By: Seluj78 <jlasne@student.42.fr>
@Seluj78
Copy link
Collaborator Author

Seluj78 commented Jan 11, 2019

@awecx @JulienPalard une grosse partie de using/windows a été changée avec le merge upstream. Je vais tout re relire et je vous ping une fois que c'est bon

@JulienPalard
Copy link
Member

@Seluj78 la CI þ

@JulienPalard
Copy link
Member

Mergé et relu sur la branche 3.7.

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.

4 participants