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

Skip to content

Traduction FR de la bibliothèque library/bz2.po #818

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 35 commits into from
Feb 1, 2020
Merged

Traduction FR de la bibliothèque library/bz2.po #818

merged 35 commits into from
Feb 1, 2020

Conversation

hucste
Copy link
Contributor

@hucste hucste commented Jun 12, 2019

No description provided.

@hucste hucste mentioned this pull request Jun 12, 2019
Closed
@Seluj78 Seluj78 added awaiting changes NEED REVIEWER This PR needs a reviewer :) and removed NEED REVIEWER This PR needs a reviewer :) awaiting changes labels Jun 15, 2019
christopheNan
christopheNan suggested changes Jun 16, 2019
View reviewed changes
@JulienPalard JulienPalard added awaiting changes and removed NEED REVIEWER This PR needs a reviewer :) labels Jul 3, 2019
@Seluj78
Copy link
Collaborator

Seluj78 commented Oct 9, 2019

@hucste ping sur les derniers conflicts ?

christopheNan
christopheNan previously approved these changes Oct 13, 2019
View reviewed changes
Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

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

Très bon travail, merci !

@JulienPalard
Copy link
Member

Quelques erreurs remontées pas hunspell :

library/bz2.po:26:bzip2.
library/bz2.po:74:bzip2
library/bz2.po:253:multiflux.
library/bz2.po:332:multiflux
library/bz2.po:452:multiflux.

JulienPalard
JulienPalard requested changes Oct 13, 2019
View reviewed changes
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.

rest syntax.

JulienPalard
JulienPalard requested changes Oct 22, 2019
View reviewed changes
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.

Remontée par travis aussi, et qq soucis de correcteur d'orthographe:

$ pospell -p dict -l fr_FR **/*.po
library/bz2.po:26:bzip2.
library/bz2.po:74:bzip2
library/bz2.po:253:multiflux.
library/bz2.po:332:multiflux
library/bz2.po:452:multiflux.

en effet, multiflux n'a pas l'air d'exister. pour bzip2 tu peux l'ajouter au dictionnaire ou le mettre en italique dans ta traduction.

christopheNan
christopheNan reviewed Oct 29, 2019
View reviewed changes
Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

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

Ok pour laisser « multiflux » qui semble être entré dans le langage courant/commercial, mais entre astérisques.
Juste une dernière remarque :

hucste
hucste commented Nov 7, 2019
View reviewed changes
Copy link
Contributor Author

@hucste hucste left a comment

Choose a reason for hiding this comment

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

Ces changements ont été pris en compte, directement par moi, après explications sur le propos. cf la dernière PR

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 15, 2019

Hello @hucste ! :D Il te reste quelsques petits changements a integret, une fois fait ping nous et opn relis/merge !

@hucste
Copy link
Contributor Author

hucste commented Nov 16, 2019

Hello @hucste ! :D Il te reste quelsques petits changements a integret, une fois fait ping nous et opn relis/merge !

Je viens de le faire.

@hucste hucste requested a review from JulienPalard November 16, 2019 13:32
@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 16, 2019

@hucste Top ! Il faut que tu mette les suggestions appliquées en resolues :)

@Seluj78 Seluj78 added NEED REVIEWER This PR needs a reviewer :) and removed awaiting changes labels Nov 16, 2019
@hucste
Copy link
Contributor Author

hucste commented Nov 16, 2019

@hucste Top ! Il faut que tu mette les suggestions appliquées en resolues :)

C'est bon, là ?!

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 16, 2019

@hucste Il y a encore des suggestions ouvertes avec des discussions en cours :) Je vais relire ta PR en attendant 👌

@hucste
Copy link
Contributor Author

hucste commented Nov 16, 2019

@hucste Il y a encore des suggestions ouvertes avec des discussions en cours :) Je vais relire ta PR en attendant ok_hand

OK, à-priori, il en restait deux, que je viens de passer en résolue ! :p

Merci de ton soutien. J'apprécie ! 👍

Seluj78
Seluj78 previously approved these changes Nov 16, 2019
View reviewed changes
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.

Allez les dernières !

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 17, 2019

@hucste on y est presque 👌 il te manque la CI ! Il faut que tu relance un powrap et tu as des balises qui foirent :)

@hucste
Copy link
Contributor Author

hucste commented Nov 18, 2019

la CI

???
Pour le reste, je viens de faire un git pull localement sur ma station…
et je vais faire le reste :p

@hucste
Copy link
Contributor Author

hucste commented Nov 18, 2019

@hucste on y est presque ok_hand il te manque la CI ! Il faut que tu relance un powrap et tu as des balises qui foirent :)

Sauf que le powrap qui foire par le Travis CI, il foire sur un fichier que je n'ai pas modifié !!!

hucste and others added 16 commits February 1, 2020 15:50
@hucste @christopheNan @JulienPalard
Update library/bz2.po
063ce6d 
Co-Authored-By: Christophe Nanteuil <[email protected]>
@hucste @christopheNan @JulienPalard
Update library/bz2.po
b9d315f 
Co-Authored-By: Christophe Nanteuil <[email protected]>
@hucste @christopheNan @JulienPalard
Update library/bz2.po
60eaa59 
Co-Authored-By: Christophe Nanteuil <[email protected]>
@hucste @JulienPalard
Corrections selon proposition de juin
080451a
@hucste @JulienPalard
Correction erreur sur :meth:
eb640bb
@hucste @JulienPalard
Correction pour 'code source'
805cc14
@hucste @JulienPalard
Fix: :meth
7725f94
@hucste @JulienPalard
Fix: mise en iitalic de termes bzip, multiflux
8f5db86
@hucste @JulienPalard
Fix : correction grammaticale
3245b54
@hucste @Seluj78 @JulienPalard
Update library/bz2.po
b05833a 
OK

Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
@hucste @Seluj78 @JulienPalard
Update library/bz2.po
212df3f 
OK

Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
@hucste @Seluj78 @JulienPalard
Update library/bz2.po
c0f451f 
OK

Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
@hucste @Seluj78 @JulienPalard
Update library/bz2.po
550babc 
OK

Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
@hucste @Seluj78 @JulienPalard
Update library/bz2.po
10c5f1d 
OK

Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
@hucste @Seluj78 @JulienPalard
Update library/bz2.po
b7a1d33 
OK

Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
@hucste @Seluj78 @JulienPalard
Update library/bz2.po
5781b70 
OK

Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
@JulienPalard JulienPalard dismissed stale reviews from Seluj78 and christopheNan via 5781b70 February 1, 2020 14:53
@JulienPalard JulienPalard changed the base branch from 3.7 to 3.8 February 1, 2020 14:53
@JulienPalard
Copy link
Member

J'ai rebasé la branche sur la 3.8.

@JulienPalard JulienPalard reopened this Feb 1, 2020
JulienPalard
JulienPalard previously approved these changes Feb 1, 2020
View reviewed changes
@JulienPalard JulienPalard merged commit ee5a8a0 into python:3.8 Feb 1, 2020
@JulienPalard
Copy link
Member

@hucste merci pour cette PR :) La CI passait super proprement une fois rebasée sur la 3.8, cool !

@hucste hucste deleted the lib-bz2 branch February 1, 2020 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deronnax deronnax deronnax left review comments

@christopheNan christopheNan christopheNan left review comments

@JulienPalard JulienPalard JulienPalard approved these changes

@Seluj78 Seluj78 Seluj78 left review comments

Assignees

@hucste hucste

@christopheNan christopheNan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@hucste @Seluj78 @JulienPalard @vpoulailleau @christopheNan @deronnax