Skip to content

Conversation

rhysd
Copy link
Contributor

@rhysd rhysd commented Jun 24, 2016

I fixed Crystal implementation for Crystal 0.18.4.

I found that a test case for time_ms failed because it requires 64bit integer. So I changed underlying type of int to Int64. Now it works.

@rhysd
Copy link
Contributor Author

rhysd commented Jun 26, 2016

Crystal used in CI looks too old (version 0.10.0). Could you tell me how to fix this? Is Version of crystal fixed?

@kanaka
Copy link
Owner

kanaka commented Jun 27, 2016

For now the docker images are under my docker hub account so I have to push them. I've rebuilt the image to pull in 0.18.4 and I've pushed it out. I restarted the test and now your changes pass.

@kanaka kanaka merged commit a95fd72 into kanaka:master Jun 27, 2016
@rhysd
Copy link
Contributor Author

rhysd commented Jun 27, 2016

@kanaka Thank you 😄

micfan pushed a commit to micfan/make-a-lisp that referenced this pull request Nov 8, 2018
luelista pushed a commit to luelista/mal that referenced this pull request Mar 10, 2024
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