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 de Sphinx et sphinx_rtd_theme #6162

Merged
merged 2 commits into from
Sep 21, 2021
Merged

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Sep 12, 2021

Mise à jour de Sphinx en 4.2.0 et sphinx_rtd_theme en 1.0.0

Je suis reparti d'une installation fraîche de Sphinx pour avoir les dernières versions des fichiers doc/Makefile, doc/make.bat et doc/source/conf.py puis j'ai adapté la configuration. Dans cette configuration, je n'ai pas remis les extensions Sphinx que nous n'avons jamais utilisé, mais elles peuvent être remises très simplement plus tard si besoin.

QA :

  • source zdsenv/bin/activate && make update && make generate-doc
  • Ouvrir doc/build/html/index.html dans votre navigateur préféré
  • Comparer la documentation générée avec la documentation actuelle sur https://docs.zestedesavoir.com

@Situphen Situphen added the C-Docs Concerne la documentation label Sep 12, 2021
@coveralls
Copy link

coveralls commented Sep 12, 2021

Coverage Status

Coverage remained the same at 86.621% when pulling 9813e4d on Situphen:sphinx into 59cadd1 on zestedesavoir:dev.

@AmauryCarrade AmauryCarrade added the En attente de la dépendance Ticket bloqué en attente d'un correctif dans une dépendance utilisée label Sep 12, 2021
@Situphen Situphen removed the En attente de la dépendance Ticket bloqué en attente d'un correctif dans une dépendance utilisée label Sep 14, 2021
@Situphen Situphen marked this pull request as ready for review September 14, 2021 20:23
Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

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

QA OK ✔️

Tout marche bien. C'est sympa d'avoir désormais les éléments d'interface en français dans la doc. ^^

J'attends que coveralls.io finisse sa maintenance pour rebaser/fusionner, je n'ai pas trop envie d'avoir la CI de la branche principale dans le rouge.

@Situphen Situphen enabled auto-merge (rebase) September 21, 2021 17:15
@Situphen Situphen merged commit 11bbf8e into zestedesavoir:dev Sep 21, 2021
@Situphen Situphen deleted the sphinx branch September 21, 2021 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Docs Concerne la documentation
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants