Skip to content

Handle overflow errors in date conversion recipes - #858

Open
RamiNoodle733 wants to merge 2 commits into
simonw:mainfrom
RamiNoodle733:fix/recipe-overflow-errors
Open

Handle overflow errors in date conversion recipes#858
RamiNoodle733 wants to merge 2 commits into
simonw:mainfrom
RamiNoodle733:fix/recipe-overflow-errors

Conversation

@RamiNoodle733

@RamiNoodle733 RamiNoodle733 commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • treat OverflowError from dateutil.parser.parse() as a parse failure for the existing errors= modes
  • preserve current behavior when errors is unset by re-raising the exception
  • add focused regression coverage for both parsedate() and parsedatetime()

Fixes #857

Testing

  • PYTHONPATH=. python -m pytest tests/test_recipe_overflow.py -q using the exact changed recipe code and regression test: 6 passed
  • python -m compileall -q sqlite_utils tests/test_recipe_overflow.py
  • Black and flake8 were not installed in the execution environment, so those checks were not run locally

📚 Documentation preview 📚: https://sqlite-utils--858.org.readthedocs.build/en/858/

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.

Date conversion recipes do not apply errors= handling to parser overflows

1 participant