-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add support for clojurescript #8
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 looks good, but i am on vacation til sunday so will merge it when I get back
Thanks! I'll start working on Infix to see if I can port it to clojurescript as well. |
It builds on travis ok, but locally it fails for me running
|
I'll take a look. |
I've uploaded 0.2.5 onto clojars, since it builds OK on travis with cloverage, but would be good to get to the bottom of this cljs error |
I've got some warnings, but no errors.
What is your java Version? |
... could explain it! |
Wow, living on the edge! I swapped from 8 to 9 and it sill works. |
yeah, ... let's not spend any more effort on this, and tentatively put it down to a java 10 incompatibility somewhere in the stack. Does seem to pop up occasionally elsewhere though: |
As foreshadowed in rm-hull/infix#28, this should make
jasentaa
clojurescript compatible.The doc files are no longer a direct copy of the worked examples, but I wasn't sure if it was worth the additional visual clutter (reader conditionals) so I left them as they were.