rust-lang/this-week-in-rust

Email publishing codegen script is not formatting correctly

Open

#3.591 geöffnet am 26. Aug. 2022

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)HTML (1.010 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (1.932 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1T 17h) (113 gemergte PRs in 30 T)

Beschreibung

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'

Contributor Guide

Research-Richtung
Das Problem liegt im This Week in Rust Repository, speziell in pelicanconf.py, wo die Umgebungsvariable TWIR NEWSLETTER THEME überprüft wird. Die Behebung beinhaltet sicherzustellen, dass das Skript die Theme Einstellung beim Generieren der E Mail korrekt erfasst. Die aktuelle Problemumgehung, das Theme direkt auf 'themes/newsletter' zu setzen, deutet auf eine einfache Codeänderung hin, um die Umgebungsvariable korrekt auszulesen und das richtige Theme anzuwenden. Es wurden keine verknüpften PRs oder Fragen von Maintainern gefunden.
Tech Stack
pythonhtmlcssshell
Domain
content
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Keine
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70