Skip to content

Conversation

@solomon23
Copy link

fixes #48
if max is less than the min, it'll loop twice. first to the end of the range from min, and than from the low end of the range to max.

I wasn't sure how you were generating the code, so i just modified the later.js directly. not sure if that's correct.

@ilanbiala
Copy link
Contributor

+1 for getting this merged in..

@bunkat
Copy link
Owner

bunkat commented Oct 13, 2014

Code needs to be fixed before it can be merged. Instead of modifying the build files, the source files need to be modified and then built. If anybody wants to update the pull request I can merge it.

@solomon23
Copy link
Author

i updated it .. #74

@ilanbiala
Copy link
Contributor

@solomon23 can this PR be closed and #74 be used?

@solomon23
Copy link
Author

yea

@solomon23 solomon23 closed this Feb 10, 2015
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.

wrapping months doesn't appear to work correctly

3 participants