-
Notifications
You must be signed in to change notification settings - Fork 102
Wrap raw HTML in StringIO #726
Copy link
Copy link
Labels
Effort > Brief 🐇Small tasks expected to take a few hours up to a couple of days.Small tasks expected to take a few hours up to a couple of days.Great First Contribution! 🌱Beginner friendly tickets with narrow scope and huge impact. Perfect to join our community!Beginner friendly tickets with narrow scope and huge impact. Perfect to join our community!Impact > Minor 🔷Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).
Metadata
Metadata
Assignees
Labels
Effort > Brief 🐇Small tasks expected to take a few hours up to a couple of days.Small tasks expected to take a few hours up to a couple of days.Great First Contribution! 🌱Beginner friendly tickets with narrow scope and huge impact. Perfect to join our community!Beginner friendly tickets with narrow scope and huge impact. Perfect to join our community!Impact > Minor 🔷Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).
pd.read_html()no longer accepts raw HTML input. You'll have to wrap it around StringIO:https://pandas.pydata.org/docs/dev/whatsnew/v3.0.0.html#other-removals
This leads to the following test failure:
chainladder-python/chainladder/core/tests/test_triangle.py
Lines 14 to 18 in 6ea3577
In this case, I think it's just part of the test setup, so it can be changed there.