Improve cljs require support. Add cljs tests

oakes / odoyle-rules

A rules engine for Clojure(Script)

The Unlicense

530 stars 20 forks source link

Improve cljs require support. Add cljs tests #5

Closed davesann closed 3 years ago

davesann commented 3 years ago

changed rules.cljc to allow clients to use basic require instead of require-macros and reader conditionals
moved rules-test.clj -> rules-test.cljc and generalised for cljs testing
added :test-cljs alias -- to test run clj -M:test-cljs
added cljs-test-runner-out to .gitignore

oakes commented 3 years ago

This looks great, thanks. I don't think the :require-macros in rules.cljc is necessary though -- I just tested it without that and cljs built fine. I'll remove it after merging but if you think i'm wrong lemme know and i'll add it back

oakes commented 3 years ago

Nevermind, i was wrong, i'll leave the :require-macros there. Thanks again.