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

Support date/times separated by a comma + Python 3.11, 3.12, and 3.13 support #203

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

Conversation

thehale
Copy link

@thehale thehale commented Nov 21, 2024

I needed this package for some projects this past week, so I took the time to upgrade it a few Python versions and add support for another date format I ran into.

Uses the github/gitignore template for Python
It took me some time/research to figure out these commands. Writing up my findings to make life easier for future me and future other contributors.
There's no need to change the `date_string` arg during the method. Avoiding doing so enables adding additional parse attempt steps.
While `dateutil` might support both booleans being set independently,
this library only supports one flag enabled at a time. As a result,
there's no sense keeping the dual flags.
Let's test how `datefinder` works on the latest stable versions of Python
As of Python 3.12, `setuptools` is no longer installed by default (see: python/cpython#95299)

At some point I'll migrate this project to the latest approaches for package management, but for now, this is a functional stopgap.
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.

1 participant