Skip to content
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

Howto unicode 3, suite et fin #873

Merged
merged 5 commits into from Sep 16, 2019
Merged

Howto unicode 3, suite et fin #873

merged 5 commits into from Sep 16, 2019

Conversation

@deronnax
Copy link
Collaborator

deronnax commented Sep 5, 2019

C'est bien francisé : guillemets français et espace insécables partout où il faut.

@deronnax
Copy link
Collaborator Author

deronnax commented Sep 5, 2019

Alors la CI a calé sur

howto/unicode.po:987:little-endian
274howto/unicode.po:987:le-endian
275howto/unicode.po:987:big-endian
276howto/unicode.po:1028:'hui
277howto/unicode.po:1108:décodables
278howto/unicode.po:1287:von
279howto/unicode.po:1287:Reedy

Je vais rajouter von et Reedy dans le dict
Le 'hui, je comprends pas 🤔. Pareil pour le-endian. Les endiants, on veux vraiment traduire par petits/gros boutistes ? (je sais que c'est le bon terme mais c'est moooooche). Décodables, c'est français, valide et courant, c'est un peu grave qu'il ne soit pas dans les dictionnaires de base, non ? Je peux le rajouter dans dict mais ça mériterait une correction upstream, non ?

Voila voila.
Bises de Martinique : https://photos.app.goo.gl/oXDEak2CfNZyuTvf7

Copy link
Collaborator

christopheNan left a comment

Merci beaucoup pour ton travail.
Juste quelques reformulations.

howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
deronnax and others added 2 commits Sep 6, 2019
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@deronnax deronnax force-pushed the deronnax:howto_unicode_3 branch from 3dc39b0 to b6de625 Sep 8, 2019
@deronnax
Copy link
Collaborator Author

deronnax commented Sep 8, 2019

Il ne reste plus comme erreur Travis que :

$ pospell -p dict -l fr_FR **/*.po
howto/unicode.po:1029:'hui
The command "pospell -p dict -l fr_FR **/*.po" exited with 255.
@deronnax
Copy link
Collaborator Author

deronnax commented Sep 10, 2019

Copy link
Collaborator

christopheNan left a comment

C'est bizarre que « aujourd'hui » de la ligne 1029 coince pour pospell car il est utilisé en ligne 59 !
Vérifie que l'apostrophe est le bon caractère.

howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
howto/unicode.po Outdated Show resolved Hide resolved
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@deronnax
Copy link
Collaborator Author

deronnax commented Sep 12, 2019

non l'apostrophe est bonne. J'ai trouvé la raison: pospell cale à cause du Aujourd'hui qui commence par une majuscule. Y'a une correction à faire à pospell je pense :D

@deronnax deronnax force-pushed the deronnax:howto_unicode_3 branch from f40f517 to 73a4c42 Sep 12, 2019
@JulienPalard JulienPalard reopened this Sep 16, 2019
@JulienPalard
Copy link
Member

JulienPalard commented Sep 16, 2019

Je viens de release pospell v0.3.10, le souci était du au double espace entre les deux phrases qui feintait la regex de detection de début de phrase. Corriger ça nous a permi de trouver une autre faute d'orthographe :)))

@JulienPalard JulienPalard reopened this Sep 16, 2019
@JulienPalard JulienPalard reopened this Sep 16, 2019
@JulienPalard JulienPalard merged commit f7829d2 into python:3.7 Sep 16, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
JulienPalard added a commit that referenced this pull request Sep 16, 2019
@JulienPalard
Copy link
Member

JulienPalard commented Sep 16, 2019

J'ai forward-porté en 3.8.(8e9cd1e).

@deronnax deronnax deleted the deronnax:howto_unicode_3 branch Sep 16, 2019
@deronnax
Copy link
Collaborator Author

deronnax commented Sep 16, 2019

J'ai forward-porté en 3.8.(8e9cd1e).

arf, du coup c'est ton nom qui apparait dans le git blame, voleur :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.