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

Allow empty strings in datetime intervals #157

Merged
merged 1 commit into from
Feb 6, 2023

Conversation

gadomski
Copy link
Member

@gadomski gadomski commented Feb 2, 2023

Related issues

Description

Adds support for empty strings in datetime intervals, e.g. 2023-02-02/. Includes tests.

I'm opening this as a draft PR b/c I'm not sure of the correct process to propogate these changes to the actual files that get run (e.g.) when you do scripts/test. To test locally, I made a new version w/ my modifications. Does every PR get a new version?

@bitner
Copy link
Collaborator

bitner commented Feb 2, 2023

Thanks @gadomski!

Every release gets a new version as each release needs to have a migration generated for it. I'm refactoring some things for 0.7.0 that will make running tests between releases much more easy (right now, you have to have a migration in order to run the tests). If you mark this as ready to review, I'll merge it and make sure it gets incorporated into the 0.7.0 release.

@gadomski gadomski marked this pull request as ready for review February 2, 2023 16:07
@bitner bitner merged commit f42e233 into main Feb 6, 2023
@gadomski gadomski deleted the allow-empty-datetime-intervals branch February 6, 2023 20:54
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.

2 participants