Closed olidroide closed 3 years ago
Hey @olidroide thanks so much for bringing this to my attention 😄! lxml is actually already listed on requirements.txt, this was an issue with it not being included on setup.py as a dependency so PyPI wasn't installing it. To avoid an extra external dependency I switched the features='lxml'
on BeautifoulSoup instantiations to features='html.parser'
so we can have more of a batteries-included approach.
Thanks again for bringing this to my attention! 👍
oups @chris-greening sorry to add again the requirement ;) great update to fix this issue :+1: thanks for all
Required lxml lib by beautifulSoup parser initialization in https://github.com/chris-greening/instascrape/blob/d1700be941355096dc59376c65769789fcb5fe5e/instascrape/core/_static_scraper.py#L240
Steps to reproduce
Issue
44