Skip to content

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

Merged
christopheNan merged 1 commit into
python:3.9from
dmerejkowsky:verbose-sphinx
Apr 30, 2021
Merged

Makefile: don't run sphinx in quiet mode#1611
christopheNan merged 1 commit into
python:3.9from
dmerejkowsky:verbose-sphinx

Conversation

@dmerejkowsky

Copy link
Copy Markdown
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
Copy Markdown

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

@JulienPalard JulienPalard left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown

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

@dmerejkowsky

Copy link
Copy Markdown
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
@dmerejkowsky dmerejkowsky dismissed stale reviews from ghost and JulienPalard via 1e3ea69 April 28, 2021 19:28
@christopheNan

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

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

Pareil :)

@christopheNan christopheNan left a comment

Copy link
Copy Markdown
Contributor

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.

4 participants