feat/lxml #5181

Merged

tobiasd merged 13 commits from feat/lxml into master 2025-07-29 08:18:01 +00:00

Conversation 1 Commits 13 Files Changed 14

+130 -123

delliott commented 2025-07-28 16:43:16 +00:00

Member

Copy Link

Copy Source

#5146 attempt 2, texts and topbanner issue resolved.

After the issues I have deployed this to https://test.fsfe.org, so we can see if there are further issues there.

It seems not though.

https://git.fsfe.org/FSFE/fsfe-website/pulls/5146 attempt 2, texts and topbanner issue resolved. After the issues I have deployed this to https://test.fsfe.org, so we can see if there are further issues there. It seems not though.

🚀 1

delliott added 13 commits 2025-07-28 16:43:18 +00:00

build: force use of system python from nix shell ... e3747cada0

solves an ssl certs issue with nltk

refactor: _build_xmlstream now lxml based 5bf6a9e9d4

feat: use lxml for all parts of press_file a727824b10

fix: do not wipe nltk_data to avoid unnecessary site hits c3199c24a1

fix: python tooling 1542558cec

fix: nix env 5e19db1a89

fix: link fixing regexes b9fd65483b

refactor: use one parser for all parsing in process file 20b84e849b

fix: also strip comments f247e41ef5

refactor: use utf8 for all xsl outputs, and disable indenting 8894ba7231

chore: ruff fixes e8b2cadde1

fix: urls b9d4cbc296

fix: put texts and topbanner in sub elements properly 7bd89852d4

delliott requested review from sofiaritz 2025-07-28 16:43:39 +00:00

delliott requested review from tobiasd 2025-07-28 16:45:05 +00:00

dmarzal commented 2025-07-28 16:48:02 +00:00

Contributor

Copy Link

Copy Source

For the podcast feed, some tweaking is still needed:

https://validator.w3.org/feed/check.cgi?url=https%3A%2F%2Ftest.fsfe.org%2F%2Fnews%2Fpodcast.en.rss

VS production:

https://validator.w3.org/feed/check.cgi?url=https%3A%2F%2Ffsfe.org%2F%2Fnews%2Fpodcast.en.rss

For the podcast feed, some tweaking is still needed: https://validator.w3.org/feed/check.cgi?url=https%3A%2F%2Ftest.fsfe.org%2F%2Fnews%2Fpodcast.en.rss VS production: https://validator.w3.org/feed/check.cgi?url=https%3A%2F%2Ffsfe.org%2F%2Fnews%2Fpodcast.en.rss

tobiasd merged commit f08d5ef925 into master 2025-07-29 08:18:01 +00:00

tobiasd referenced this issue from a commit 2025-07-29 08:18:03 +00:00

feat/lxml (#5181)

bonnie referenced this pull request 2025-07-31 08:38:40 +00:00

SFP:absolute URLs for Podcast RSS Feed #5190

Sign in to join this conversation.

Reviewers

No Reviewers

sofiaritz

tobiasd

Labels

Clear labels

bug

build

cgi Scripting

design

disruptive

Requires careful coordination and documentation changes

documentation

duplicate

easy

enhancement

Improving what is already there

feature-request

help wanted

javascript

priority/low

question

system-hackers

tagging

text

translations

wait/bugfix

wait/inprogress

wait/misc

wait/proofread

wontfix

xsl

No Label

Milestone

No items

No Milestone

Assignees

Clear assignees

albert (Albert Dengg) alex.busch (alex.busch) alex.sander (alex.sander) anaghz (anaghz) annarita.russo (annarita.russo) ao (André Ockers) bcludts (bcludts) bonnie (bonnie) ciampix (Marco Ciampa) cryptie (cryptie) dario (dario) delliott (delliott) dfajfer (Damian Fajfer) dmaphy (Dominic Hopf) doczkal (Thomas Doczkal) eal (eal) egnun (Erik Grun) eventregbot (eventregbot) fi (fi) floriansnow (Florian Snow) gabriel.ku (gabriel.ku) guido (guido) hugo (hugo) ineiev (ineiev) jithendra (jithendra) jn (jn) jzarl (jzarl) linus (Linus Sehn) lucabon (Luca Bonissi) lucas.lasota (lucas.lasota) max.mehl (Max Mehl) mk (Matthias Kirschner) monochromec (monochromec) mweimann (Michael Weimann) nico.rikken (nico.rikken) patrick (Patrick Ohnewein) reinhard (Reinhard Müller) renovate-bot (Renovate Bot) repentinus (Heiki Lõhmus) schiessle (Björn Schießle) silviarbgl (Silvia Rbgl) sofiaritz (Sofía Aritz) tobiasd (tobiasd) vincent (vincent)

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: FSFE/fsfe-website#5181