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

Use native toISOString when we can #2009

Closed
wants to merge 1 commit into
base: develop
from

Conversation

Projects
None yet
2 participants
@elliotf
Contributor

elliotf commented Oct 24, 2014

The main idea is to take advantage of the speed in the native .toISOString when possible.

The node tests pass and I hope that there is CI coverage of browsers. Please let me now if you'd like anything more or if you'd like anything different.

Related to issue #1999

@ichernev ichernev closed this Nov 17, 2014

ichernev added a commit that referenced this pull request Nov 17, 2014

@elliotf

This comment has been minimized.

Contributor

elliotf commented Nov 17, 2014

Was there a problem with this pull request?

@ichernev

This comment has been minimized.

Contributor

ichernev commented Nov 17, 2014

I'm merging it manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment