Skip to content
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

Fix UserWarning in prettify.py #244

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ciscoqid
Copy link

@ciscoqid ciscoqid commented Oct 6, 2023

When using the prettify post-processor, this is emitted:

...rss2email-3.14-py3.6.egg/rss2email/post_process/prettify.py:85: UserWarning: No parser was explicitly specified, so I'm using the best available HTML parser for this system ("lxml"). This usually isn't a problem, but if you run this code on another system, or in a different virtual environment, it may use a different parser and behave differently.

The code that caused this warning is on line 85 of the file ...rss2email-3.14-py3.6.egg/rss2email/post_process/prettify.py. To get rid of this warning, pass the additional argument 'features="lxml"' to the BeautifulSoup constructor.

When using the prettify post-processor, this is emitted:

...rss2email-3.14-py3.6.egg/rss2email/post_process/prettify.py:85: UserWarning: No parser was explicitly specified, so I'm using the best available HTML parser for this system ("lxml"). This usually isn't a problem, but if you run this code on another system, or in a different virtual environment, it may use a different parser and behave differently.

The code that caused this warning is on line 85 of the file ...rss2email-3.14-py3.6.egg/rss2email/post_process/prettify.py. To get rid of this warning, pass the additional argument 'features="lxml"' to the BeautifulSoup constructor.
@auouymous
Copy link
Contributor

So we must require html5lib or lxml, or have it choose a parser and throw a warning?

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