-
Notifications
You must be signed in to change notification settings - Fork 76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Initial support for Italian publishers, starting with La Repubblica #670
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ruggsea Thanks a lot for pushing Italian publishers forward. That's a great addition and your code looks good! Just some minor adjustments.
…h to get topics, modified the dynamic sitemap handling to conform to tagesspiel implementation, removed redundant free access check
…h to get topics, modified the dynamic sitemap handling to conform to tagesspiel implementation, removed redundant free access check
Thank you for your feedback. I have implemented your advice in my latest commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ruggsea Huge thanks for adding this :)
Add La Repubblica (Italian News)
Hey! I've added support for La Repubblica, one of Italy's leading newspapers. I would like this project to cover Italian newspapers too and I figured I should start from this one
Additions
What can it parse?
The parser can extract:
Missing
Testing
Created and passed unit tests; run script to add La Repubblica to the Supported Sources docs. Run linting and mypy checking as required