-
Notifications
You must be signed in to change notification settings - Fork 1k
time: Add time module to provide strftime. #508
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
Conversation
FYI I've also got a recent open PR to add a It'd certainly be trivial to merge them eventually. It's probably worth you adding |
I don't think it should import everything in there. |
In micropython, the C implementation of builtins are in the If there's a python / frozen version of the non-u version provided however that overrides the shadow; the intention is to allow python modules to extend the built-ins exactly like this PR is doing. As-is, once this time.py is added to path,
That way |
Ah okay, but you said :
I think you meant from |
Ah geez sorry for the confusion, I missed the unwanted autocorrect! |
d7df55c
to
a0ee55c
Compare
Signed-off-by: Damien George <damien@micropython.org>
Thanks for updating. Merged, and I removed the I also added a unit test (taking out the test from |
No description provided.