Skip to content

Commit

Permalink
Reverted lxml dependency removal.
Browse files Browse the repository at this point in the history
The lxml dependency is a bs4 dependency. But we must require it.
I think using `beautifulsoup4[lxml]` is the best, but don't know how
to test it.
On Debian this is forced by python3-bs4, but when using pip the lxml
dependency is optional.
Sorry for the noise.
  • Loading branch information
set-soft committed Mar 10, 2021
1 parent e892d37 commit ae8a4d7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
beautifulsoup4 >= 4.3.2 # Deal with HTML and XML tags.
#lxml >= 3.7.2
lxml >= 3.7.2 # Used by beautifulsoup4, but optional.
XlsxWriter >= 0.7.3 # Write the XLSX output file.
future # For print statements.
tqdm >= 4.30.0 # Progress bar.
Expand Down

0 comments on commit ae8a4d7

Please sign in to comment.