rust-lang/this-week-in-rust

Email publishing codegen script is not formatting correctly

Open

#3 591 ouverte le 26 août 2022

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)HTML (1 010 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (1 932 stars)
Métriques de merge PR
 (Merge moyen 1j 17h) (113 PRs mergées en 30 j)

Description

The Pelican publishing script used by the Makefile command optimize-email is not picking up TWIR_NEWSLETTER_THEME setting and defaulting to a CSS style that does not match our most recent issues (e.g. 454).

Currently, the workaround for this has been to change pelicanconf.py from:

if '1' == os.environ.get('TWIR_NEWSLETTER_THEME'):
    THEME = 'themes/newsletter'
else:
    THEME = 'themes/rusted'

to:

THEME = 'themes/newsletter'

Guide contributeur

Direction de recherche
Le problème se trouve dans le dépôt This Week in Rust, spécifiquement dans pelicanconf.py où la variable d'environnement TWIR NEWSLETTER THEME est vérifiée. La correction implique de s'assurer que le script récupère correctement le paramètre de thème lors de la génération de l'e mail. Le contournement actuel qui consiste à définir directement le thème sur 'themes/newsletter' suggère un simple changement de code pour lire correctement la variable d'environnement et appliquer le thème approprié. Aucune PR liée ou question de mainteneur n'a été trouvée.
Stack technique
pythonhtmlcssshell
Domaine
content
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70