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 timezone awareness under Python 2.7.x #33

Merged
merged 1 commit into from
Mar 23, 2020

Conversation

aaron-sierra
Copy link
Contributor

I noticed the build is failing under Python 2.7.x as a result of the changes I introduced in #32...

The datetime module doesn't provide timezone.utc under Python 2.7.x, so
define it for ourselves.
@MartijnBraam
Copy link
Owner

I'd hate to break it to you... but python 2.7 is EOL

@aaron-sierra
Copy link
Contributor Author

I'd hate to break it to you... but python 2.7 is EOL

Don't feel bad about saving me work, unless you mean that you've officially dropped Python 2.7 support in this module. Builds are still happening against Python 2.7 and the README still has instructions for installing for it, too.

I didn't want to be the reason that Python 2.7 stopped being supported.

@MartijnBraam
Copy link
Owner

Well the change to keep 2.7 supported in this module seems relatively minor. It's probably a good idea to keep support around for that since this code is already written now :)

@MartijnBraam MartijnBraam merged commit 831b827 into MartijnBraam:master Mar 23, 2020
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.

None yet

2 participants