Skip to content

Conversation

@Kydlaw
Copy link
Contributor

@Kydlaw Kydlaw commented Nov 23, 2019

Closes #1023
Closes #1021
Closes #1030
Closes #1034

Copy link
Contributor

@vpoulailleau vpoulailleau left a comment

Choose a reason for hiding this comment

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

Super ! ❤️

Quelques petites remarques en vrac et en vitesse… (Désolé pour l’absence de rédaction de mes remarques…)

Copy link
Collaborator

@Seluj78 Seluj78 left a comment

Choose a reason for hiding this comment

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

Super travail ! Quelques commentaires 🤗

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 25, 2019

PS, c'est la PR 1024. Coïncidence ? Je ne crois pas...

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 25, 2019

 Reviews #1 Co-Authored-By: Jules Lasne (jlasne) <jlasne@student.42.fr> Co-Authored-By: Vincent Poulailleau <vpoulailleau@gmail.com>
@vpoulailleau
Copy link
Contributor

à rajouter: pypi.org/project/poutils

Je viens d'aller voir sur PyPI, il n'y a que la licence dans la description du projet 😉 mais en regardant le setup.cfg ça peut effectivement simplifier l'installation de tous ces outils (je m'étais posé la même question, et je voulais faire la même chose mais je n'ai pas pris le temps de le faire). Au passage, il faudrait penser à dire les versions d'outils installés dans le setup.cfg, non ?

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.

Merci pour cette PR.

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 26, 2019

Je viens d'aller voir sur PyPI, il n'y a que la licence dans la description du projet 😉

Fuck ! tu peux me faire une issue ?

Au passage, il faudrait penser à dire les versions d'outils installés dans le setup.cfg, non ?

Tu pense qu'il ne vaux pas mieux demander la latest ?

Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com> Co-Authored-By: Julien Palard <julien@palard.fr> Co-Authored-By: Vincent Poulailleau <vpoulailleau@gmail.com>
@vpoulailleau
Copy link
Contributor

Je viens d'aller voir sur PyPI, il n'y a que la licence dans la description du projet wink

Fuck ! tu peux me faire une issue ?

AFPy/poutils#1

Au passage, il faudrait penser à dire les versions d'outils installés dans le setup.cfg, non ?

Tu pense qu'il ne vaux pas mieux demander la latest ?

Non je ne pense pas, j'ai détaillé mon point de vue dans AFPy/poutils#2, libre à toi ensuite de le prendre en compte ou non 😉

@Kydlaw
Copy link
Contributor Author

Kydlaw commented Nov 29, 2019

J'ai finis de prendre en compte les remarques et les modifications.
@JulienPalard j'ai pris en compte tes commits et j'ai résolus les conflits.

Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
- Un compte `Github <https://github.com/join>`_.
- Un client ``git`` `Linux <https://git-scm.com/>`_ ou `Windows <https://gitforwindows.org/>`_.
https://gitforwindows.org/).
- Un éditeur de fichier ``.po`` (comme `poedit <https://poedit.net/>`_).
Copy link
Collaborator

Choose a reason for hiding this comment

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

préciser les que un éditeur de texte suffit mais pas recommandé ? et que des plugins existent pour emacs (https://www.gnu.org/software/gettext/manual/html_node/PO-Mode.html) et vim ?


Pour commencer vous aurez besoin de *forker* le dépôt des sources `python-docs-fr
<https://github.com/python/python-docs-fr>`_ en cliquant sur son bouton
``Fork``. Ceci crée une copie du projet sur votre compte Github : un endroit
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
``Fork``. Ceci crée une copie du projet sur votre compte Github : un endroit
``Fork``. Ceci crée une copie du projet sur votre compte Github : un endroit
Copy link
Collaborator

Choose a reason for hiding this comment

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

cc @awecx, les espaces insécables ;)

# Ici, remplacez « library/sys.po » par le fichier que vous avez choisi précédemment.
poedit library/sys.po
# Si vous n'utilisez pas poedit, vous pouvez utiliser `powrap` (voir la section *outils*)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Peut être une fois mergé


Outils utiles pour la traduction
--------------------------------

Copy link
Collaborator

Choose a reason for hiding this comment

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

@Seluj78
Copy link
Collaborator

Seluj78 commented Dec 5, 2019

Hello @Kydlaw ! Peux tu rajouter qq part cette video ? https://youtu.be/azXmvpEJMhU

Ca fermera la #1034

@JulienPalard
Copy link
Member

Je merge, car c'est franchement bien, et qu'il y aura toujours des détails à retoucher mais qu'on peut les faire dans d'autres PRs. GG @Kydlaw.

@JulienPalard JulienPalard merged commit e39a69c into python:3.8 Dec 5, 2019
@Kydlaw
Copy link
Contributor Author

Kydlaw commented Dec 6, 2019

Cool !
Merci pour le merge !

@christopheNan
Copy link
Contributor

Merci pour cette très bonne initiative !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

6 participants