Skip to content
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

lein-cljsbuild warnings #9

Closed
murtaza52 opened this issue Jun 3, 2012 · 2 comments
Closed

lein-cljsbuild warnings #9

murtaza52 opened this issue Jun 3, 2012 · 2 comments

Comments

@murtaza52
Copy link

Hi,

The lein-cljsbuild auto gives me the following warnings, I am not sure if these are something that need to be fixed, so just bringing to attention.

However, Thanks for the great library!

WARNING: Wrong number of args (1) passed to cljs.core.instance_QMARK_ at line 262 file:/home/murtaza/.m2/repository/enfocus/enfocus/0.9.1-SNAPSHOT/enfocus-0.9.1-SNAPSHOT.jar!/enfocus/core.cljs
WARNING: Use of undeclared Var enfocus.core/op-handler at line 438 file:/home/murtaza/.m2/repository/enfocus/enfocus/0.9.1-SNAPSHOT/enfocus-0.9.1-SNAPSHOT.jar!/enfocus/core.cljs

@superbobry
Copy link
Contributor

Hi there, I've already fixed that in my domina-migration branch, so I guess this'll be fixed as soon as @ckirkendall merges this with upstream ;)

@ckirkendall
Copy link
Owner

There are lot more warning on the domina branch now most around domina vars not enfocus. I moved over the base css-select and content over to domina and checked it in. The are some gotchas related to the domina selector that took quite a bit to iron out. The biggest is it is lazy evaluated each time you call nodes. This could be an issue given the dynamic nature of the dom. I am going to dicuss it with Luke to see if I can get it moved to eager eval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants