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

add nginx-clojure support #160

Merged
merged 3 commits into from
Sep 3, 2015
Merged

Conversation

xfeep
Copy link
Collaborator

@xfeep xfeep commented Aug 31, 2015

Add a nginx-clojure adapter which can work with nginx-clojure v0.4.2+.

{:status 200 :body nc-ch})))

(def nginx-clojure-adapter (NginxClojureAsyncNetworkChannelAdapter.))
(def sente-web-server-adapter nginx-clojure-adapter) ; Alias for ns import convenience
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No carriage return at end of file

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danielcompton Thank you! I have added a newline.

@ptaoussanis
Copy link
Member

Hi Zhang, this looks great - thank you! 👍

Just a bit busy to look at this in detail today, will try get it merged in the next day or two!

Cheers :-)

@xfeep
Copy link
Collaborator Author

xfeep commented Sep 1, 2015

@ptaoussanis No rush :-)

ptaoussanis added a commit that referenced this pull request Sep 3, 2015
@ptaoussanis ptaoussanis merged commit 414957d into taoensso:master Sep 3, 2015
xfeep added a commit that referenced this pull request Sep 3, 2015
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