rubys / feedvalidator

A validator for syndicated feeds. It works with Atom, RSS feeds as well as OPML and KML formats.
http://feedvalidator.org/
Other
115 stars 42 forks source link

Python 3 #46

Open mnot opened 3 years ago

mnot commented 3 years ago

I looked into what it would take to convert to 3, since 2 is no longer supported.

The short list is:

Where I got stuck was in how the code uses object inheritance so extensively; thanks to the switch to new-style objects, this doesn't work any more, and the trivial fix (removing things like validatorBase to avoid duplication) seems to mess with the processing model, causing many errors. E.g.,

TypeError: Cannot create a consistent method resolution
order (MRO) for bases text, rfc2396_full

Fixing this looks pretty invasive, and I'm not confident enough in this codebase to make the necessary changes. If someone is interested in helping out, I can do a PR for the things above and work with them on the solution.

rubys commented 3 years ago

Go for it! I will try to answer any questions, and may even make some fixes.