-
Notifications
You must be signed in to change notification settings - Fork 21
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
Support for Clojurescript #3
Comments
FWIW, reagent-project/reagent-template includes it but I can't tell if it actually works... |
👍 |
I'd like to see this work, if it is possible. humane-test-output does Clojure specific not-so-nice things to |
I started looking into it. It sort of works (there are just some excessive newlines...) And everything is duplicated into cljs (for now). currently it is required to require some additional stuff in tests like do-report: 80f9137...nenadalm:clojurescript_support#diff-20936c87d473fea7e5d6351a664d091fR3 as I can't figure how to do that without these additional stuff here: I am starting with clojure/clojurescript - so if anybody is willing to help... I also included some files into $ lein test | diff test/fixtures/test_output -
$ lein cljsbuild test 2>&1 | diff test/fixtures/test_output_cljs - |
Now files are merged and excessive requires are removed. Except I accidentaly created 2 files in namespace pjstadig.macro and when renaming the on from test/ it stops working. |
@nenadalm would you mind creating a PR so we can start a conversation around your changes. Thanks for taking this on! |
done. np (it doesn't work anyway yet) |
Clojurescript now has cljs.test. It'd be great if we can get the same awesome test output there :)
The text was updated successfully, but these errors were encountered: