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

date parsing is broken on linux #1

Closed
wants to merge 1 commit into from
Closed

Conversation

woloski
Copy link

@woloski woloski commented Jul 9, 2013

This patch creates a thin wrapper in js and will do the parsing there instead.

@Southern
Copy link
Owner

Thanks so much for reporting this bug. I've pushed a native fix to the master branch, rather than depending on Javascript dependencies to do it. I also bumped the version to 0.0.2 and published it to the NPM registry. Give it a test run and let me know what happens.

I tested it on Ubuntu 13.04. If it has problems, please let me know what operating system you're using so I can test on it instead.

@Southern
Copy link
Owner

@woloski Everything working okay in the new versions?

@Southern
Copy link
Owner

Been almost a week with the new release. Going to close this for now. Reopen it if you're still experiencing problems. :)

@Southern Southern closed this Jul 17, 2013
@woloski
Copy link
Author

woloski commented Jul 17, 2013

did not have time to try it out really. thanks for the quick fix!

On Wednesday, 17 de July de 2013 at 11:14, Colton Baker wrote:

Been almost a week with the new release. Going to close this for now. Reopen it if you're still experiencing problems. :)


Reply to this email directly or view it on GitHub (#1 (comment)).

@Southern
Copy link
Owner

No problem. :)

yorkie added a commit that referenced this pull request Sep 14, 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