Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

màj: utilisation de htmlserve au lieu de serve dans le Makefile#1919

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

eviau
Copy link

@eviaueviau commentedNov 5, 2022
edited
Loading

En lien avec#1918 .

Je ne suis pas 100% convaincue que c'est la façon de faire ce changement... Suggestions (et explications) bienvenues.

Avecmake htmlview, je réussi à générer la doc localement mais je ne vois pas comment accéder à la version traduite... j'ai regardé sur d'autres repos de doc python -exemple - et je ne vois pas trop ce que je dois changer.

C'est probablement pas le moment de faire ce changement - je vous laisse en juger :)

@JulienPalard
Copy link
Member

Attentionhtmlview etserve ne font pas du tout la même chose :htmlview fait unbuild puis lance un serveur HTTP alors que serve ne fait que lancer un serveur HTTP.

Mais c'est vrai qu'htmlview est sympa.

Je serai plutôt d'avis d'ajouter une règlehtmlview plutôt que de modifier la règleserve.

Comme ça unmake serve chez nous fait unmake serve chez cpython, on ne casse les habitudes de personne.

Et unmake htmlview chez nous fait unmake htmlview sur cpython : c'est prévisible.

Pour implémenter la règlehtmlview tu peux simplement changer le mode de la règleall:

.PHONY: htmlviewhtmlview: MODE=htmlviewhtmlview: all

car j'ai peur que comme tu l'as implémenté tu te retrouves avec la doc en anglais ?

christopheNan reacted with thumbs up emoji

@eviau
Copy link
Author

D'accord - oui en effet, tel que je l'avais fait dans le premier commit, je n'avais que la version en anglais.

Maintenant je peux fairemake htmlview et avoir la doc en français ✨

Je ne sais pas niveau historique git si ce que j'ai fait fonctionne ? Et niveau Makefile ?

Dites-moi s'il y a quoique ce soit :)

@JulienPalard
Copy link
Member

Niveau historique git ne t'en fais pas je ferai un "squash" avant de merger pour qu'il n'y en ai qu'un.

Ton commit est bon. Vu que tu viens de me faire découvrir qu'upstream ils ont retirémake serve, tu peux aussi retirermake serve chez nous, il ne fonctionne plus de toute façons...

Il faudra que tu mette à jour leCONTRIBUTING.rst pour refléter ce changement aussi.

@eviau
Copy link
Author

Voilà - je ne pense pas avoir rien oublié... ?

@JulienPalard
Copy link
Member

Je ne suis pas encore utilisateur d'htmlview, mais je crois que dans contributing l'étape :

Une fois la traduction finie, il faut compiler la documentation,c'est-à-dire générer les fichiers HTML affichés par le site, pour lesrelire. Si la commande précédente s'est exécutée sans erreur, lacompilation ne devrait pas échouer...code-block::bash    make

est à merger avec l'étape suivante ?

Si j'ai bien compris, avant on devait fairemake suivi demake serve, alors que maintenant un seulmake htmlview suffit ?

@christopheNan
Copy link
Contributor

Je ne suis pas encore utilisateur d'htmlview, mais je crois que dans contributing l'étape :

Une fois la traduction finie, il faut compiler la documentation,c'est-à-dire générer les fichiers HTML affichés par le site, pour lesrelire. Si la commande précédente s'est exécutée sans erreur, lacompilation ne devrait pas échouer... code-block:: bash    make

est à merger avec l'étape suivante ?

Si j'ai bien compris, avant on devait fairemake suivi demake serve, alors que maintenant un seulmake htmlview suffit ?

Oui, make htmlview fait les 2. Mais il bien écrit quemake vérifie si la compilation passe etmake htmlview permet de voir le rendu dans le navigateur (et fait la compilation au besoin).

@JulienPalard
Copy link
Member

Oui, make htmlview fait les 2. Mais il bien écrit que make vérifie si la compilation passe et make htmlview permet de voir le rendu dans le navigateur (et fait la compilation au besoin).

Il faut que je teste, ce dont j'ai peur c'est que lemake puismake htmlview prenne presque deux fois plus longtemps quemake +make serve oumake htmlview seul.

@christopheNan
Copy link
Contributor

christopheNan commentedNov 13, 2022 via email

AMHA, non, il detecte bien que la compilation est propreLe dim. 13 nov. 2022, 15:26, Julien Palard ***@***.***> aécrit :
Oui, make htmlview fait les 2. Mais il bien écrit que make vérifie si la compilation passe et make htmlview permet de voir le rendu dans le navigateur (et fait la compilation au besoin). Il faut que je teste, ce dont j'ai peur c'est que le make puis make htmlview prenne presque deux fois plus longtemps que make + make serve ou make htmlview seul. — Reply to this email directly, view it on GitHub <#1919 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AILBNUBYBVH7V4XBGGV2Z4LWID3CZANCNFSM6AAAAAARXWBCJA> . You are receiving this because you commented.Message ID: ***@***.***>

@JulienPalard
Copy link
Member

AMHA, non, il detecte bien que la compilation est propre

Je viens de tester, 1m12 pour lemake et3s pour lemake htmlview c'est propre ♥

@JulienPalardJulienPalard merged commitda21f1d intopython:3.11Nov 13, 2022
@JulienPalard
Copy link
Member

Merci@eviau !

christopheNan reacted with thumbs up emojieviau reacted with hooray emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@christopheNanchristopheNanchristopheNan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@eviau@JulienPalard@christopheNan

[8]ページ先頭

©2009-2025 Movatter.jp