-
Notifications
You must be signed in to change notification settings - Fork 9
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 shadow-cljs #3
Comments
Will take this on, thinking of getting CI done first then can do this and add it to CI |
I've started running the CLJS tests and looking at the errors. I'm seeing an out of memory exception during the compilation phase, however. Is this expected? Note, ignore the error in
|
First, the tests are compiling as they have to be compiled to a node js script to run at all. That error is partially expected. Encountered it locally and in ci, but was hoping that was a side effect of all the broken tests. Anyway, I'll modify the list of tests it runs and pinpoint the one spiking the memory. |
Yea, there is a particular assertion that is causing the issue. It's this one:
The system doesn't seem to like the infinite sequence generated by |
We want to run these tests with both CLJ and CLJS. To do so, we need to bring in shadow-cljs. We'll also need to tweak some of the tests to use reader conditionals, particularly around
(thrown? Foo ...)
forms.The text was updated successfully, but these errors were encountered: