I think it is needed and I have included a unit test that should fail
without it if my assumption is correct. Think about what set difference
does...
On Mar 21, 2013 8:36 PM, "AlexBaranosky" <notifications@github.com> wrote:

> In src/clj_clu/core.clj:
>
> > +  (format "missing required arguments: %s\n"
> > +          (str/join ", " (map name missing))))
> > +
> > +(defn ^:private required-option? [option]
> > +  (-> (drop-while (partial not= :required) option) next first))
> > +
> > +(defn ^:private required-options [options]
> > +  (keep (fn [op] (when (required-option? op)
> > +                   (-> (take-while #(= \- (first %)) op)
> > +                       last
> > +                       (str/replace #"^-*" "")
> > +                       keyword)))
> > +        options))
> > +
> > +(defn ^:private missing-options [actual required]
> > +  (seq (set/intersection required (set/difference required actual))))
>
> I think maybe this set/intersection call is not needed.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/runa-dev/clj-clu/pull/1/files#r3484133>
> .
>