Simplify kmg.domain/recommendation-ids

As we all know, parentheses are evil. Let's get rid of some of them and make the code more readable!
Oops, sorry. clojure.core/remove is the answer :)
On Jul 9, 2014 9:07 PM, "Alexander Petrov" <notifications@github.com> wrote:

> In src/kmg/domain.clj:
>
> > @@ -56,7 +56,7 @@
> >  (defn recommendation-ids [db user]
> >    (let [recs (recommendations-for-user db user)
> >          completed (set (recommendations-completed-by-user db user))]
> > -    (filter #(not (contains? completed %)) recs)))
> > +    (keep completed recs)))
>
> Unfortunately these two functions not equivalent. This function means that
> I want to get recommendations which is not completed yet. But (keep
> completed recs) just keep exactly completed recommendations, which is
> opposite to what is required. I definitely need to add tests for all domain
> logic:)
> Anyway, thank you for try!
> May be there is another way to simplify this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alexpetrov/kmg/pull/1/files#r14725543>.
>