search

zestedesavoir / Python-ZMarkdown

NOT USED ANYMORE. Small fork of original Python-Markdown with modifications suitables for Zeste de Savoir (original : https://github.com/waylan/Python-Markdown )
Other
12 stars 10 forks source link

Nouvelle extension pour la typographie française. #102

Closed DominusCarnufex closed 7 years ago

DominusCarnufex commented 8 years ago

Il s’agit d’une reprise de la PR https://github.com/zestedesavoir/Python-ZMarkdown/pull/76 concernant le nouveau module de typographie française. Par rapport à la PR précédente, voici les changements.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.1%) to 93.999% when pulling 3cc3d063ecc9e3edf0129b13fbb38a81f6b8737a on DominusCarnufex:typographie into 0fd12b1e8abb7cc8b98f87d13f5ea560a0608740 on zestedesavoir:master-zds.

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.1%) to 93.999% when pulling 3cc3d063ecc9e3edf0129b13fbb38a81f6b8737a on DominusCarnufex:typographie into 0fd12b1e8abb7cc8b98f87d13f5ea560a0608740 on zestedesavoir:master-zds.

cgabard commented 8 years ago

Il est apparu un bogue que je ne parviens pas à corriger : le smiley :'( n’est plus reconnu.

Ton extension devrait passer après celle des émoticons, donc ça devrait fonctionner. Je vais regarder quand j'aurais un peu de temps

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.2%) to 94.08% when pulling 10887ffc5021c1c707b29574ea5b33b3aeb2220e on DominusCarnufex:typographie into 0fd12b1e8abb7cc8b98f87d13f5ea560a0608740 on zestedesavoir:master-zds.

DominusCarnufex commented 8 years ago

Ouais, ça doit être lié à ma manière de faire les tests, j’ai d’autres bogues bizarres qui apparaissent comme ça. En tout cas, avec les derniers ajouts, TRAVIS passe à merveille, donc tout a l’air bon (à part l’absence de tests spécifiques au module).

DominusCarnufex commented 8 years ago

Up @cgabard : du nouveau ? Qu’est-ce qui empêche encore le merge ?

dvbmgr commented 7 years ago

Ping ?

cgabard commented 7 years ago

Il faut que je termine le chantier de ménage avant (bientôt) et je ferai une passe d'ajout de ces PR.

Il y a eu

cgabard commented 7 years ago

Désolé, problème de tel...

Bref au passage j'aurais bien aimé que ça soit vite fait commenté sur le site (pas à en faire une zep mais au moins un sujet) juste pour être sûrs qu'il n'y a pas d'objection et de défaut auxquels on aurait pas pensé

dvbmgr commented 7 years ago

Oui enfin sans vouloir être méchant, ça fait 3 mois qu'il n'y a pas eu de commits ici.

Il y aura des objections parce que des gens vont pas comprendre, et ça va prendre trois mois à être mergé.

Les problèmes que ça peut introduire sont clairs: si quelqu'un s'amuse à mettre du code dans un message sans les balises ça va pas être beau à voir. Si quelqu'un copie du code qui a été écrit dans un message sans balises, ça va pas être beau à voir. Si quelqu'un poste un lien sans mettre la balise de lien, ça va pas être beau à voir.

Mais à un moment, il faut arrêter de prendre les utilisateurs pour des idiots, ou alors qu'ils ne se plaignent pas de l'être.

Et comme discuté, cela force à respecter les conventions typographiques établies sur le site. Il n'y a pas à discuter davantage.

cgabard commented 7 years ago

Who on s'énerve pas...

Si tu veux reprendre la maintenance et le dev du bouzin et tout ce que ça implique, vas y.

Ensuite :

Si quelqu'un poste un lien sans mettre la balise de lien, ça va pas être beau à voir.

Ça je peux pas accepter car le zmarkdown actuellement l'autorise explicitement donc si ça pète, c'est problématique !

Et comme discuté, cela force à respecter les conventions typographiques établies sur le site. Il n'y a pas à discuter davantage.

Ça a été discuté ou ça, déjà ? Ensuite je demande pas de faire une zep hein, juste que ce soit clairement dit quelque part. Sinon je peux aussi appliquer strictement la règle actuel : "pas de changement de fonctionnalités tant qu'on a pas tout réorganisé le code et que les pdf sont nickel" mais je suis pas sûrs que tu préfère cette solution.

Le mer. 2 nov. 2016 10:04, David Baumgartner notifications@github.com a écrit :

Oui enfin sans vouloir être méchant, ça fait 3 mois qu'il n'y a pas eu de commits ici.

Il y aura des objections parce que des gens vont pas comprendre, et ça va prendre trois mois à être mergé.

Les problèmes que ça peut introduire sont clairs: si quelqu'un s'amuse à mettre du code dans un message sans les balises ça va pas être beau à voir. Si quelqu'un copie du code qui a été écrit dans un message sans balises, ça va pas être beau à voir. Si quelqu'un poste un lien sans mettre la balise de lien, ça va pas être beau à voir.

Mais à un moment, il faut arrêter de prendre les utilisateurs pour des idiots, ou alors qu'ils ne se plaignent pas de l'être.

Et comme discuté, cela force à respecter les conventions typographiques établies sur le site. Il n'y a pas à discuter davantage.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/zestedesavoir/Python-ZMarkdown/pull/102#issuecomment-257808432, or mute the thread https://github.com/notifications/unsubscribe-auth/AF0jfP-deKpubNCDIeGiD-x3NKHDkMI_ks5q6FIJgaJpZM4JEST1 .

dvbmgr commented 7 years ago

Je ne m'énerve pas, je fais juste remarquer qu'il n'y a pas eu de commits depuis trois mois et que rien n'avance. Et je trouve ça problématique.

Ça je peux pas accepter car le zmarkdown actuellement l'autorise explicitement donc si ça pète, c'est problématique !

Alors il manque un test explicite pour ça (probablement qu'il faut compléter tests/zds/rediger_sur_zds_part3.txt), mais cela fonctionne comme attendu.

Ça a été discuté ou ça, déjà ? Ensuite je demande pas de faire une zep

https://zestedesavoir.com/contenus/beta/1027/conventions-typographiques-en-usage-sur-zeste-de-savoir/

fonctionnalités tant qu'on a pas tout réorganisé le code et que les pdf sont nickel" mais je suis pas sûrs que tu préfère cette solution.

Oui enfin il y a bien des chances que ZMarkdown passe à la trappe pour générer les PDFs à terme. Ça me semble assez suicidaire de tout « refactoriser » (=tout réécrire) alors qu'il manque de dévs et qu'il y a probablement des outils qui le feraient très bien sans nécessiter ce travail.

cgabard commented 7 years ago

Il n'y en a pas eu sur master car ce sur quoi je travaille quand j'ai le temps demande plusieurs modifs un peu partout (doit y avoir une branche sur le dépôt plus récente déjà).

Pour le pdf on va pas refaire le débat ici.

De mon côté je vais finir le nettoyage des extensions et tests (j'arrive à la fin) et la suite prévu était de merger ces nouvelles fonctionnalités (celle-ci, le ping et celle de firm1 avec quelques ajustements)

Le mer. 2 nov. 2016 10:56, David Baumgartner notifications@github.com a écrit :

Je ne m'énerve pas, je fais juste remarquer qu'il n'y a pas eu de commits depuis trois mois et que rien n'avance. Et je trouve ça problématique.

Ça je peux pas accepter car le zmarkdown actuellement l'autorise explicitement donc si ça pète, c'est problématique !

Alors il manque un test explicite pour ça (probablement qu'il faut compléter tests/zds/rediger_sur_zds_part3.txt), mais cela fonctionne comme attendu.

Ça a été discuté ou ça, déjà ? Ensuite je demande pas de faire une zep

https://zestedesavoir.com/contenus/beta/1027/conventions-typographiques-en-usage-sur-zeste-de-savoir/

fonctionnalités tant qu'on a pas tout réorganisé le code et que les pdf sont nickel" mais je suis pas sûrs que tu préfère cette solution.

Oui enfin il y a bien des chances que ZMarkdown passe à la trappe pour générer les PDFs à terme. Ça me semble assez suicidaire de tout « refactoriser » (=tout réécrire) alors qu'il manque de dévs et qu'il y a probablement des outils qui le feraient très bien sans nécessiter ce travail.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/zestedesavoir/Python-ZMarkdown/pull/102#issuecomment-257819979, or mute the thread https://github.com/notifications/unsubscribe-auth/AF0jfHEF5p6WfXULmZwWi8RQz471dmzxks5q6F5NgaJpZM4JEST1 .

GerardPaligot commented 7 years ago

Je viens juste mettre un lien ici entre cette PR ci-présente et cette issue puisque cette PR pourrait bien casser les smileys. C'est à tester.

DominusCarnufex commented 7 years ago

Bonsoir les gens.

Alors pour répondre aux remarques et interrogations diverses…

Cette extension typographique, ça fait quasiment un an et demi qu’elle est en discussion sur le forum. La première proposition a été faite dans ce sujet mais sans que ce soit suivi d’effet. Elle a été remise au goût du jour dans ce sujet qui est, pour partie, à l’origine de la ZEP-41. Quant à la définition exacte des règles à respecter, elle est évidemment donnée dans les conventions typographiques.

Elle ne casse pas les liens libres : elle n’opère que si les deux-points et le pour-cent sont précédés d’une espace, ce qui n’est pas le cas dans une URL.

Elle ne casse pas non plus les smileys, j’y ai veillé. Elle agit après l’extension qui gère les smileys, donc pas de problème. Si jamais l’extension des smileys venait à se retrouver après celle de typographie, seul le smiley :'( serait impacté, à cause de l’apostrophe : les deux-points et point-virgules ne sont affectés que s’ils sont suivis d’une espace ou d’une fin de ligne. Ainsi, : D est affecté, mais pas :D.

De manière plus générale, cette extension a été conçue de manière à ne pas interférer avec la typographie des autres langues que le français, donc elle n’affecte que des points qui sont sans aucun doute possible de la ponctuation française. Par exemple, les guillemets droits " ne sont pas transformés automatiquement, parce qu’ils pourraient se trouver dans un texte en anglais ou en allemand ; pour avoir des guillemets français sans les taper soi-même, il faut utiliser << et >>.

J’espère que cela vous rassure sur le soin apporté à cette extension. :wink:

cgabard commented 7 years ago

Si ça te dérange pas, je vais probablement m'occuper du rebase et pour faciliter ça faire ça dans une autre branche/PR

cgabard commented 7 years ago

Je ferme ici, je merge dans #118 merci @DominusCarnufex