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

add timezone support #81

Closed
wants to merge 1 commit into from
Closed

Conversation

mauro-ni
Copy link

This is related to #67

@mauro-ni mauro-ni mentioned this pull request Feb 22, 2018
@mauro-ni
Copy link
Author

@pseudomuto what do you think?

@pseudomuto pseudomuto self-assigned this Feb 22, 2018
@mauro-ni
Copy link
Author

@pseudomuto any update on this? The failing check reported by travis is due to the length of a class (too many lines).

@mauro-ni
Copy link
Author

mauro-ni commented Nov 8, 2019

@pseudomuto Any update on this?

@fgrehm
Copy link
Owner

fgrehm commented Feb 18, 2020

@maurosbu sorry for the long delay here!

I don't have a strong need for this but I'm fine with having the feature around. Would you mind updating your branch with latest master and adding some test coverage for it?

@fgrehm fgrehm added the waiting label Feb 18, 2020
@fgrehm
Copy link
Owner

fgrehm commented Oct 20, 2021

Hey @maurosbu ! I'm getting ready to cut a 2.0 release in #113 and this has become stale. We're also moving away towards just using the email headers there so things will look like this now:

image

I'm going to close it for now, feel free to open a new issue if you or anyone else think this is still valuable!

EDIT: Sorry, forgot to thanks for taking the time to write the PR and apologize for taking so long to get back to ya

@fgrehm fgrehm closed this Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants