Unpin lxml

[adamh-oai]

Per adbar/trafilatura#449, see if unpinning
lxml works on MacOS.

[adbar]

Hi @adamh-oai, the tests pass, the current issue is unrelated, a dummy site is down.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4088710) 99.07% compared to head (b8cc607) 99.07%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #114   +/-   ##
=======================================
  Coverage   99.07%   99.07%           
=======================================
  Files           8        8           
  Lines         867      867           
=======================================
  Hits          859      859           
  Misses          8        8           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[adbar]

Now it works for Python 3.12 but not for Python 3.8 on MacOS.

[mde-pach]

I ran the tests locally and everything works well with the same version of python on macos (m2 architecture)
Can this test not works because of the content of the website during the run ?
Did you try to rerun them ?

Edit: I just saw the tests are run using a x86 architecture, maybe the difference come from there

[adbar]

Yes, I updated the tests. I'm not sure why tests fail specifically on Python 3.8 but you're probably right in assuming different architectures lead to different outcomes.

We could try adding other versions of Python on MacOS, so far other architectures are not easily available on Github Actions as far as I know.

[adbar]

@adamh-oai I believe I found the right combination of settings and tests, what do you think?

[adamh-oai]

lgtm!

[adamh-oai]

Could you put up a new release on pypi with this change? Thanks!

[adbar]

Yes, I plan to do it this week for htmldate and a bit later for trafilatura.