Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix parsing exception when trailing/leading slash #505

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix parsing exception when trailing/leading slash #505

wants to merge 2 commits into from

Conversation

fffaez
Copy link

@fffaez fffaez commented Oct 16, 2020

Hello everybody, I am new here 🙂

I fix the _iso8601_interval parsing accordingly to deal with this edge case so the outer parse could raise a proper ValueError.

Fixes #506

Pull Request Check List

  • Added tests for changed code.

    • yes, just a couple
  • Updated documentation for changed code.

    • nothing to really update IMO

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.

Wrong exception when parsing string with trailing or leading slash
1 participant