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

Passe à zmarkdown 11 #6276

Merged
merged 3 commits into from
Apr 3, 2022
Merged

Passe à zmarkdown 11 #6276

merged 3 commits into from
Apr 3, 2022

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Mar 29, 2022

Passe à zmarkdown 11 (changelog)

Fixes #6275

Je propose de vérifier les modifications concernant le LaTeX (notes de bas de page et tableaux) directement sur la bêta.

Cette branche est sur la bêta

QA :

  • À partir de upstream/dev (donc avec zmarkdown 10.1.3), créer un message avec un bloc spoiler
  • source zdsenv/bin/activate && make update && make zmd-start && make run-back
  • Créer un message avec :
    • Un bloc spoiler sans titre
    • Un bloc spoiler avec titre
    • Un bloc de code d'une seule ligne
    • Un bloc de code de plusieurs lignes
    • Des pings de pseudos avec des tirets et des accents

Ce que j'ai testé :

```py
oneliner = lambda x: x**2
```

```py
def twoliner(x):
    return x**2
```

Bonsoir, comment allez-vous ?[^note]

[^note]: Super Note de bas de page !

[[secret|Attention, ceci contient des mots de passe très secrets !]]
| - **Platine**
| - Or
| - *Argent*
| - ~~Bronze~~

[[secret]]
| - **Platine**
| - Or
| - *Argent*
| - ~~Bronze~~

@Bidule @Super-Bidule @Bidule-Hypé @Bidule-Ultra-Trop-Éclairé-de-la-vie @**Bidule**

@coveralls
Copy link

coveralls commented Mar 29, 2022

Coverage Status

Coverage remained the same at 87.202% when pulling 3980837 on Situphen:zmd-11 into 9ddac71 on zestedesavoir:dev.

@Situphen
Copy link
Member Author

Situphen commented Mar 29, 2022

Après des tests sur la bêta, je confirme que les failles de sécurité sont bien corrigées. Les notes de bas de page aussi.

Par contre la génération du PDF du tuto C++ échoue, à cause d'un timeout j'ai l'impression. Idem pour celui sur le C.

@Situphen
Copy link
Member Author

La génération des PDF des gros contenus comme celui sur le C++ ou sur le C échoue car on dépasse la limite des 150 Mo par worker zmarkdown. Après avoir augmenté cette limite à 300 Mo, on dépasse la limite timeout de 120 secondes côtés zds-site. Augmenter cette limite à 300 secondes, ce qui est énorme, ne résout pas le soucis.

@Situphen
Copy link
Member Author

J'ai mis à jour la PR avec zmd 11.0.1 et j'ai mis cette branche sur la bêta. La génération du LaTeX fonctionne correctement pour les gros contenus désormais. La génération des PDF fonctionne correctement pour Arduino et Langage C. Elle échoue pour La programmation en C++ mais cela ne me semble pas venir de zmd 11, donc je pense qu'on peut fusionner cette PR.

Un premier message de la bêta pour vérifier que les blocs de code, les blocs secrets et les pings fonctionnent correctement avec zmd 11. Un deuxième message de la bêta pour vérifier que les anciens blocs secrets fonctionnent correctement.

@philippemilink
Copy link
Member

Elle échoue pour La programmation en C++ mais cela ne me semble pas venir de zmd 11

Qu'est-ce qui te fais dire que ça ne vient pas de zmd 11 ? Sur la bêta la compilation échoue, mais sur la prod elle réussit...

@philippemilink
Copy link
Member

Elle échoue pour La programmation en C++ mais cela ne me semble pas venir de zmd 11

Qu'est-ce qui te fais dire que ça ne vient pas de zmd 11 ? Sur la bêta la compilation échoue, mais sur la prod elle réussit...

J'ai identifié le problème et ouvert une issue chez zmd : zestedesavoir/zmarkdown#461. Je n'ai pas vérifié que le problème ne se produit pas avec zmd10, mais les fichiers LaTeX du tuto C++ générés sur la prod et la bêta sont différent au niveau qui pose problème, donc ça me semble tout de même être ça ;)

@StaloneLab
Copy link
Member

Problème résolu normalement dans la version 11.0.2 de ZMd.

@Situphen
Copy link
Member Author

Situphen commented Apr 2, 2022

C'est sur la bêta et le PDF du tuto C++ est bien généré !

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

Rapport de QA

  • Testé avec l'exemple proposé avant/après la mise à jour de zmd sur le même sujet de forum (ainsi l'ancien et le nouveau bloc secret sont sur la même page) : OK
  • Le problème de note de bas de page dans un titre dans le PDF du tutoriel C++ est corrigé (cf sur la bêta).

OK ✔️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Mise à jour de ZMarkdown vers la version 11
4 participants