[PR #13] [CLOSED] Fix invalid XML #15

New Issue

GiteaMirror · 2025-11-06T16:19:31-06:00

GiteaMirror commented

2025-11-06 16:19:31 -06:00

📋 Pull Request Information

Original PR: https://github.com/plenaryapp/awesome-rss-feeds/pull/13
Author: @ralexander-phi
Created: 5/21/2024
Status: ❌ Closed

Base: master ← Head: master

📝 Commits (2)

0416162 Fixing broken XML in OPML files
57418ee Clean up OPML formatting

📊 Changes

118 files changed (+1809 additions, -1825 deletions)

View changed files

📝 countries/with_category/Australia.opml (+18 -18)
📝 countries/with_category/Bangladesh.opml (+10 -10)
📝 countries/with_category/Brazil.opml (+9 -9)
📝 countries/with_category/Canada.opml (+12 -12)
📝 countries/with_category/France.opml (+13 -13)
📝 countries/with_category/Germany.opml (+7 -7)
📝 countries/with_category/Hong Kong SAR China.opml (+7 -7)
📝 countries/with_category/India.opml (+38 -38)
📝 countries/with_category/Indonesia.opml (+6 -6)
📝 countries/with_category/Iran.opml (+9 -9)
📝 countries/with_category/Ireland.opml (+8 -8)
📝 countries/with_category/Italy.opml (+15 -15)
📝 countries/with_category/Japan.opml (+10 -10)
📝 countries/with_category/Mexico.opml (+17 -17)
📝 countries/with_category/Myanmar (Burma).opml (+5 -5)
📝 countries/with_category/Nigeria.opml (+12 -12)
📝 countries/with_category/Pakistan.opml (+9 -9)
📝 countries/with_category/Philippines.opml (+22 -22)
📝 countries/with_category/Poland.opml (+10 -10)
📝 countries/with_category/Russia.opml (+15 -15)

...and 80 more files

📄 Description

Hey folks, awesome list!

I've also noticed issues with invalid XML in the OPML files. I've gone through each OPML file and manually fixed problems. I stripped extra quotes, converted & to &, stripped out HTML tags, and removed special characters like newlines.

I've validated the output using:

find . -name '*.opml' -print0 | xargs -0 -I '{}' xmllint --valid '{}' --noout 2>&1 | egrep -v "(no DTD)|(opml version)|(    \^)"

This ignores DTD error messages, which aren't technically validation problems.

I've also run the fix-up script in #2, which normalizes a couple minor things.

I've also noticed that many feed URLs are now returning errors. I'm considering replacing or removing those. Is this project interested in third party contributors, or would it be best for me to continue work in a fork?

Thanks for releasing this list!

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/plenaryapp/awesome-rss-feeds/pull/13 **Author:** [@ralexander-phi](https://github.com/ralexander-phi) **Created:** 5/21/2024 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`0416162`](https://github.com/plenaryapp/awesome-rss-feeds/commit/0416162780d6c27e70962015cc72d897e4daf552) Fixing broken XML in OPML files - [`57418ee`](https://github.com/plenaryapp/awesome-rss-feeds/commit/57418eeb4a091510d336fc9887e1d8b49bd8e8ed) Clean up OPML formatting ### 📊 Changes **118 files changed** (+1809 additions, -1825 deletions) <details> <summary>View changed files</summary> 📝 `countries/with_category/Australia.opml` (+18 -18) 📝 `countries/with_category/Bangladesh.opml` (+10 -10) 📝 `countries/with_category/Brazil.opml` (+9 -9) 📝 `countries/with_category/Canada.opml` (+12 -12) 📝 `countries/with_category/France.opml` (+13 -13) 📝 `countries/with_category/Germany.opml` (+7 -7) 📝 `countries/with_category/Hong Kong SAR China.opml` (+7 -7) 📝 `countries/with_category/India.opml` (+38 -38) 📝 `countries/with_category/Indonesia.opml` (+6 -6) 📝 `countries/with_category/Iran.opml` (+9 -9) 📝 `countries/with_category/Ireland.opml` (+8 -8) 📝 `countries/with_category/Italy.opml` (+15 -15) 📝 `countries/with_category/Japan.opml` (+10 -10) 📝 `countries/with_category/Mexico.opml` (+17 -17) 📝 `countries/with_category/Myanmar (Burma).opml` (+5 -5) 📝 `countries/with_category/Nigeria.opml` (+12 -12) 📝 `countries/with_category/Pakistan.opml` (+9 -9) 📝 `countries/with_category/Philippines.opml` (+22 -22) 📝 `countries/with_category/Poland.opml` (+10 -10) 📝 `countries/with_category/Russia.opml` (+15 -15) _...and 80 more files_ </details> ### 📄 Description Hey folks, awesome list! I've also noticed issues with invalid XML in the OPML files. I've gone through each OPML file and manually fixed problems. I stripped extra quotes, converted `&` to `&`, stripped out HTML tags, and removed special characters like newlines. I've validated the output using: find . -name '*.opml' -print0 | xargs -0 -I '{}' xmllint --valid '{}' --noout 2>&1 | egrep -v "(no DTD)|(opml version)|( \^)" This ignores DTD error messages, which aren't technically validation problems. I've also run the fix-up script in #2, which normalizes a couple minor things. I've also noticed that many feed URLs are now returning errors. I'm considering replacing or removing those. Is this project interested in third party contributors, or would it be best for me to continue work in a fork? Thanks for releasing this list! --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2025-11-06 16:19:31 -06:00

GiteaMirror closed this issue

2025-11-06 16:19:31 -06:00

GiteaMirror referenced this issue

2025-11-06 16:19:33 -06:00

[PR #15] [CLOSED] Update Mexico.opml #16

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/awesome-rss-feeds#15