Skip to content

removed whitespace at end of lines and added a comment about customization #3

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

arunpersaud
Copy link

tried to customize the CHARSET_LIST and only figured after a while that it's also in configy.py which overwrote any changes I did in rss2email. Added a comment to mention this in rss2email.py.

ps: would be great to have a link on http://www.allthingsrss.com/rss2email/ to the github repo...

@wking
Copy link

wking commented Jan 9, 2013

I picked up rss2email maintainership and added a PyPI page homepage that does link to my GitHub repository. I'll review new PRs there, but this PR no longer applies to my master (I restructured rss2email's configuration setup a fair bit).

@arunpersaud
Copy link
Author

On 01/09/2013 07:47 AM, W. Trevor King wrote:

I picked up rss2email maintainership and added a PyPI page
http://pypi.python.org/pypi/rss2email/ homepage that does link to my
GitHub repository https://github.com/wking/rss2email/. I'll review new
PRs there, but this PR no longer applies to my master (I restructured
rss2email's configuration setup a fair bit).

good to hear that development is continuing. I don't have much time at
the moment, but will clone your new repo and submit any future changes
via github. Still using rss2email for all my feeds ;)

Thanks

Arun

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants