Skip to content

Makefile: don't run sphinx in quiet mode #1611

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

Merged
merged 1 commit into from
Apr 30, 2021

Conversation

dmerejkowsky
Copy link
Contributor

No description provided.

@python python deleted a comment from PyDocTeur Apr 28, 2021
@python python deleted a comment from PyDocTeur Apr 28, 2021
@python python deleted a comment from PyDocTeur Apr 28, 2021
@PyDocTeur
Copy link

Hello @dmerejkowsky ! Bienvenue dans le monde merveilleux de la traduction de la documentation de Python !
Je suis PyDocTeur, le robot amical qui s'occupe d'automatiser certaines tâches.
Si tu as des questions, n'hésite pas à ping @JulienPalard ou @Seluj78


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: greetings)
PyDocTeur v1.11.1

JulienPalard
JulienPalard previously approved these changes Apr 28, 2021
Copy link
Member

@JulienPalard JulienPalard left a comment

Choose a reason for hiding this comment

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

J'ai rien contre.

@PyDocTeur
Copy link

ON Y EST PRESQUE ! Un p'tit label automerge et je merge ça !


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: approved)
PyDocTeur v1.11.1

HS-157
HS-157 previously approved these changes Apr 28, 2021
Copy link
Contributor

@HS-157 HS-157 left a comment

Choose a reason for hiding this comment

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

Idem.

@dmerejkowsky
Copy link
Contributor Author

On n'a pas trouvé pourquoi le -q était là, du coup voici une explication pour avoir sphinx en mode verbeux:

1/ parfois sphinx est lent du coup ça donne une indication de ce qu'il est en train de faire et ça évite que le traducteur/la traductrice s'ennuie
2/ si jamais y a un message de "warning" ou "erreur" il est possible d'avoir plus de contexte.

Rationale:

* Sometimes Sphinx is slow and it's nice to have progress indicators while it's
  running

* If there's a warning or error, it may be possible to get relevant context
@christopheNan
Copy link
Contributor

Il pourrait être intéressant de mettre cette option avec une variable associée sur le modèle de JOBS afin de permettre la modification en lançant le make make VERBOSE=0 (par défaut, make serait verbeux)

@JulienPalard
Copy link
Member

Il pourrait être intéressant de mettre cette option avec une variable associée sur le modèle de JOBS afin de permettre la modification en lançant le make make VERBOSE=0 (par défaut, make serait verbeux)

Yep, si on en ressent vraiment le besoin, on l'ajoutera, d'ici là je parie « YAGNI ».

@dmerejkowsky
Copy link
Contributor Author

Yep, si on en ressent vraiment le besoin, on l'ajoutera, d'ici là je parie « YAGNI ».

Pareil :)

Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

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

🚀

@christopheNan christopheNan merged commit 26361e7 into python:3.9 Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants