-
Notifications
You must be signed in to change notification settings - Fork 534
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
Error building techan.js in EDT (America/New_York) #64
Comments
Not sure what is going on there, it works for me on Yosemite. I cleaned out all dependencies and ran the same commands (apart from the clone) and it was fine. I also just kicked off another build on Travis CS which was ok too. The dates seem to be one index off expected which is strange. My setup (which I haven't touched for a while):
I'll try to execute in your timezone and see if I can get the same result. |
I can recreate the issue. I've added a Dockerfile and by enabling the lines that set the the timezone to yours the build will fail with what appear to be the same errors. docker build -t techan.js .
docker run --rm -it techan.js test Not sure what the root cause is just yet, but at least that container will allow me to play around. |
Cool, I will try building the docker file today. On Sunday, April 19, 2015, André Dumas notifications@github.com wrote:
Regards, S. Brad Sollar |
Almost have a fix. It's due to the interesting ways dates are parsed in javascript. Would always work for me because my local timezone is in the plus offset (+10) of GMT (UTC) and yours in in the negative, generally putting it a day out (after time truncation). Will run a few more tests before I push. |
Hello, I am sorry, but I am living in Germany/Europe and I have the same issue. Maybe it's because of the timezone. But because I am new to npm and similar tools: What should I do with the Dockerfile to get it running? |
I am getting this error when installing on my Mac. I updated NPM(1.4.28) and Node(v0.10.32).
Here were the commands I used;
git clone --recursive https://github.com/andredumas/techan.js.git
cd techan.js
npm install
npm start
The text was updated successfully, but these errors were encountered: