Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Restructuration du README et déplacement de certains éléments dans CO… #1024

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 11 commits into from
Dec 5, 2019
Merged

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

à rajouter: https://pypi.org/project/poutils/

Reviews #1

Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
Co-Authored-By: Vincent Poulailleau <[email protected]>
@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
Contributor

@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 <[email protected]>
Co-Authored-By: Julien Palard <[email protected]>
Co-Authored-By: Vincent Poulailleau <[email protected]>
@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 <[email protected]>
- 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
Projects
None yet
6 participants