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

⬆️ Mise à jour vers Docusaurus v3 #8

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Afizullah
Copy link

@Afizullah Afizullah commented Mar 4, 2024

Bonjour,

Merci pour ce travail, c'était très intéressant à lire.
Cette PR migre Sillon vers Docusaurus v3.
J'ai suivi le guide de migration, en particulier le point de vigilance concernant la migration vers le format MDX v3.

➜ sillon git:(main) npx docusaurus-mdx-checker
[SUCCESS] All 105 MDX files compiled successfully!

Également, un petit changement au niveau de l'API de la dépendance prism-react-renderer

Pas de soucis du côté de la génération du PDF, j'ai installé PrincePDF en locale et ait pu généré le document normalement.

Je n'ai pas remarqué de régression visuel.

Cordialement,

@sneko
Copy link
Collaborator

sneko commented Mar 4, 2024

Bonjour @Afizullah ,

Je vois dans ta PR que tu as rajouté la librairie i sans pour autant l'utiliser. Peux-tu m'en dire un peu plus ?

Autre point, le site marchant en l'état, je regarderai la PR plus en détails quand une migration sera nécessaire pour un besoin spécifique.

Merci,

@Afizullah Afizullah force-pushed the main branch 3 times, most recently from 1ea497c to 21d6ad5 Compare March 4, 2024 19:45
@Afizullah
Copy link
Author

Afizullah commented Mar 4, 2024

Bonjour,

elle était superflue a été retirée.

Autre point, le site marchant en l'état, je regarderai la PR plus en détails quand une migration sera nécessaire pour un besoin spécifique.

Pas de soucis, au moins il n'y a pas grand chose à faire si un jour le besoin s'en ressent. Je compte peut-être utiliser ce thème pour une documentation en interne à la DGFiP.

Cordialement,

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.

2 participants