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

auto should have an option to run tests as well #216

Open
mullr opened this issue Aug 4, 2013 · 9 comments · May be fixed by #222
Open

auto should have an option to run tests as well #216

mullr opened this issue Aug 4, 2013 · 9 comments · May be fixed by #222
Milestone

Comments

@mullr
Copy link

mullr commented Aug 4, 2013

lein cljsbuild auto is great! I wish it could also run tests after building.

@hugoduncan
Copy link

Seconded!

@hugoduncan hugoduncan linked a pull request Aug 17, 2013 that will close this issue
@astarr
Copy link

astarr commented Sep 11, 2013

+1 !

@cemerick
Copy link
Collaborator

I agree, and will pull in Hugo's patch as soon as v1.0 is out. I want to get that done, and don't want to load a new subtask into the mix.

@joshuafcole
Copy link

Is this patch still slated to be pulled in? It'd definitely be handy. I tried the test-refresh plugin, but it doesn't seem to play nicely with test hooks (I'm using the cljsbuild plugin and working primarily in cljs).

@cemerick
Copy link
Collaborator

@joshuafcole Yes, coming soon.

@magomimmo
Copy link
Contributor

Great! Thanks Chas.
mimmo

On Jan 25, 2014, at 11:56 AM, Chas Emerick [email protected] wrote:

@joshuafcole Yes, coming soon.


Reply to this email directly or view it on GitHub.

@ddellacosta
Copy link

+1

Would cry tears of joy if we could get Hugo's patch in. If there's anything that needs to happen before then that needs to be delegated I'm game for some cljsbuild hacking/yak-shaving.

@bellkev
Copy link

bellkev commented Aug 22, 2014

👍

1 similar comment
@AriPaaWun
Copy link

+1

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

Successfully merging a pull request may close this issue.

9 participants