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

Skip to content

Rectification des *fuzzy* du fichier sys.po. #1099

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 4 commits into from
Feb 4, 2020
Merged

Rectification des *fuzzy* du fichier sys.po. #1099

merged 4 commits into from
Feb 4, 2020

Conversation

lonis00
Copy link
Contributor

@lonis00 lonis00 commented Dec 14, 2019

Fix #1089

@JulienPalard JulienPalard self-assigned this Dec 14, 2019
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 cette PR ! Quelques remarques.

@lonis00
Copy link
Contributor Author

lonis00 commented Dec 15, 2019 via email

@JulienPalard
Copy link
Member

Ah bon ? Tant mieux et merci encore.

Oui, car une « Pull Request » c'est litéralement une demande de merger une branche. Quand tu vas git commit, git push tu vas mettre à jour ta branche. La demande de merger la branche tient toujours, la branche a juste évolué.

@lonis00
Copy link
Contributor Author

lonis00 commented Dec 15, 2019 via email

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.

Relu jusqu'a la ligne 156.

@JulienPalard
Copy link
Member

@louisMAURY Bonjour Louis.

Lorsque tu te remet à traduire comme là deux jours plus tard sur un sujet légèrement différent (ici OK c'est discutable, c'est le même fichier, mais il y a deux jours tu travaillais sur des fuzzy, et là tu travaille sur des traductions), tu devrais créer une nouvelle branche, et donc une nouvelle PR, comme ça on peux avancer sur le merge du premier travail pendant que tu es sur le second.

Pour faire une nouvelle branche, fie toi à CONTRIBUTING.rst qui dit en gros :

git fetch upstream
git checkout -b le_nom_de_ta_nouvelle_branche upstream/3,8

La première commande permet de s'assurer que tu es à jour, la 2ème fait partir ta nouvelle branche sur la version à jour de la 3.8.

Ça ne marche que si chez toi la remote upstream pointe bien sur https://github.com/python/python-docs-fr/, tu peux voir ça avec git remote -v.

Pour ce coup ci, ne change rien, ne t'embête pas, mais pour le prochain coup fais une branche.

@vpoulailleau
Copy link
Contributor

Ça ne marche que si chez toi la remote upstream pointe bien sur python/python-docs-fr, tu peux voir ça avec git remote -v

C'est effectivement le cas, puisque la recommandation pour toute nouvelle contribution est le paragraphe que tu cites « en gros », et qu'on a évoqué. Par contre j'ai pas insisté sur le fait de faire deux branches pour deux modifications non corrélées dans un seul et même fichier…

@JulienPalard
Copy link
Member

@vpoulailleau c'est pas très gênant, ça va retarder un peu le merge des fuzzy, mais y'a pas le feu au lac :)

@lonis00
Copy link
Contributor Author

lonis00 commented Dec 16, 2019 via email

@christopheNan
Copy link
Contributor

Pour t'aider, tu peux regarder l'article wikipédia de hook. Plusieurs traductions sont possibles : fonction de rappel, point d'entrée, voire procédure automatique… Tu peux varier et choisir celui qui correspond le mieux au contexte.

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.

Quelques petits soucis d'orthographe et c'est bon pour moi.

@JulienPalard JulienPalard merged commit 3dedf73 into python:3.8 Feb 4, 2020
@JulienPalard
Copy link
Member

@louisMAURY j'ai poussé ma relecture de décembre et j'ai pu merger ta branche, merci pour cette PR :)

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/sys.po
4 participants