search

nervous-systems / fink-nottle

Asynchronous Clojure/Clojurescript client for Amazon's SNS & SQS services
The Unlicense
48 stars 4 forks source link

duplicate defmethod in fink-nottle.sqs.tagged #3

Closed sirobinson closed 9 years ago

sirobinson commented 9 years ago

Should the final line here: (defmulti message-in (fn [tag body] tag)) (defmethod message-in :default [ body] body) (defmulti message-out (fn [tag body] tag)) (defmethod message-in :default [ body] body)

be (defmethod message-out .....

rather than message-in?

moea commented 9 years ago

Thanks, well spotted. I'll fix it once I've finished/released the Clojurescript port which is hanging out in master - a day or two.

sirobinson commented 9 years ago

Wow

Thanks for quick response!

From: Moe Aboulkheir notifications@github.com<mailto:notifications@github.com> Reply-To: nervous-systems/fink-nottle reply@reply.github.com<mailto:reply@reply.github.com> Date: Friday, 24 July 2015 11:56 To: nervous-systems/fink-nottle fink-nottle@noreply.github.com<mailto:fink-nottle@noreply.github.com> Cc: Simon Robinson simon.robinson@mixrad.io<mailto:simon.robinson@mixrad.io> Subject: Re: [fink-nottle] duplicate defmethod in fink-nottle.sqs.tagged (#3)

Thanks, well spotted. I'll fix it once I've finished/released the Clojurescript port which is hanging out in master - a day or two.

Reply to this email directly or view it on GitHubhttps://github.com/nervous-systems/fink-nottle/issues/3#issuecomment-124473937.

moea commented 9 years ago

Fixed in 0.4.1