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

nrepl #3

Closed
wants to merge 7 commits into from
Closed

nrepl #3

wants to merge 7 commits into from

Conversation

hiredman
Copy link

@hiredman hiredman commented Jul 4, 2012

got clojure evaluation happening via nrepl (https://github.com/clojure/tools.nrepl)

  • clojure language examples all work
  • breaks the cascalog examples for some reason, so maybe not ready for a merge yet

@kovasb
Copy link
Owner

kovasb commented Jul 4, 2012

Awesome!! I will look at it and try to merge.

Right now I'm in the process of converting everything possible to tagged literals, now that we have direct support for those in cljs. And generally cleaning up the code :)

@hiredman
Copy link
Author

hiredman commented Jul 5, 2012

I think the issue is the nrepl requests are not stateless, and so the session/cookie info is being lost across requests. There doesn't seem to be a quick fix for that

@hiredman
Copy link
Author

hiredman commented Jul 5, 2012

the very last def in the cascalog examples still fails, I think due to something about https://github.com/nathanmarz/cascalog/blob/develop/src/clj/cascalog/util.clj#L163 so not a session issue per se

@hiredman
Copy link
Author

I wrote an nrepl/drawbridge client in cljs https://github.com/hiredman/drawbridge-cljs and an nrepl middleware for compiling clojurescript https://github.com/hiredman/nrepl-cljs-middleware, with those two combined nrepl can be used to eval clojure and compile clojurescript

@kovasb
Copy link
Owner

kovasb commented Dec 12, 2012

Alright, I think I'm ready to address this now :) In the new world, we are gonna want an nrepl thing that plays the datomic transaction game. Will take a look at your projects.

@jonase jonase mentioned this pull request Jan 2, 2013
@kovasb
Copy link
Owner

kovasb commented Jan 15, 2013

we got nrepl going within the new architecture (thanks to jonase) , so closing this guy

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

Successfully merging this pull request may close these issues.

2 participants