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

strftime and tests #23

Closed
wants to merge 1 commit into from
Closed

strftime and tests #23

wants to merge 1 commit into from

Conversation

catawbasam
Copy link

Adds strftime() formatting by converting Date / Datetime to TmStruct and using TmStruct's strftime().

@quinnj
Copy link
Owner

quinnj commented Nov 27, 2013

Hey @catawbasam, thanks for the PR. I'm actually planning on doing a lot of work with this in about 2 weeks, so I'm going to let this sit until then. Hopefully that's alright. The plan is to actually merge the package into Base and I would imagine get rid of the TmStruct, so this may not be needed (as long as we have all the functionality in this package).

@catawbasam
Copy link
Author

Hi Jacob,
Ok. If there's anything I can do to help, just let me know.
cheers,
Keith

On Tue, Nov 26, 2013 at 8:28 PM, Jacob Quinn [email protected]:

Hey @catawbasam https://github.com/catawbasam, thanks for the PR. I'm
actually planning on doing a lot of work with this in about 2 weeks, so I'm
going to let this sit until then. Hopefully that's alright. The plan is to
actually merge the package into Base and I would imagine get rid of the
TmStruct, so this may not be needed (as long as we have all the
functionality in this package).


Reply to this email directly or view it on GitHubhttps://github.com//pull/23#issuecomment-29353213
.

@quinnj
Copy link
Owner

quinnj commented Aug 26, 2014

Sorry I never got back to you on this. The Dates.jl package (and now Dates module in Base) has much more sophisticated formatting capabilities, so hopefully that suits the functionality you were looking for. If not, I'm happy to help continue working on it (e.g. we just added the ability to put the day of the week in a formatted datetime to conform with HTTP protocols).

@quinnj quinnj closed this Aug 26, 2014
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