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

Effort to dynamically reload html #9

Closed
wants to merge 58 commits into from

Conversation

jeroenvandijk
Copy link

Open issue: The HEAD request doesn't seem to be catched by compojure; not when you do a request via curl and not when live.js tries to it.

See #8

@jeroenvandijk
Copy link
Author

Shouldn't be merged btw :-)

@zcaudate
Copy link
Member

zcaudate commented May 1, 2014

Thanks Jeroen!

It's a really great idea and removes the need for external applications.

I'm actually on holidays atm and don't have my computer with me... but will be back in 3 weeks time and will be able to play with it then.

My main problem with the leiningen plugin is that is too slow for me. Its nice to have things run straight away without waiting for the clojure runtime to warm up. What I really want to do is to port rewrite-clj to clojurescript. Its been on the back of my mind for a while - clj-commons/rewrite-clj#4 (comment) and having looked more closely at the code, it shouldn't be too hard to do.

The midje-doc parser and renderer should be directly portable (it needs to be rewritten anyways) and then an express server could be spun up with no java dependencies :)

On 01/05/2014, at 9:20, Jeroen van Dijk notifications@github.com wrote:

Open issue: The HEAD request doesn't seem to be catched by compojure; not when you do a request via curl and not when live.js tries to it.

See #8

You can merge this Pull Request by running

git pull https://github.com/jeroenvandijk/lein-midje-doc feature/live-js
Or view, comment on, or merge it at:

#9

Commit Summary

WIP
File Changes

M project.clj (6)
A resources/public/live.js (233)
M src/leiningen/midje_doc/run.clj (35)
M test/midje_doc/my_first_document.clj (2)
Patch Links:

https://github.com/zcaudate/lein-midje-doc/pull/9.patch
https://github.com/zcaudate/lein-midje-doc/pull/9.diff

Reply to this email directly or view it on GitHub.

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 this pull request may close these issues.

None yet

3 participants