is it new? we need this references because of the if block.


On Sun, Mar 30, 2014 at 3:17 AM, Haim Yadid <notifications@github.com>wrote:

> In src/main/java/com/performizeit/mjstack/MJStack.java:
>
> > @@ -17,34 +17,46 @@
> >
> >  package com.performizeit.mjstack;
> >
> > -import com.performizeit.mjstack.api.JStackFilter;
> > -import com.performizeit.mjstack.api.JStackMapper;
> > +import static com.performizeit.mjstack.monads.StepProps.CONTAINS;
> > +import static com.performizeit.mjstack.monads.StepProps.COUNT;
> > +import static com.performizeit.mjstack.monads.StepProps.ELIMINATE;
> > +import static com.performizeit.mjstack.monads.StepProps.GROUP;
> > +import static com.performizeit.mjstack.monads.StepProps.KEEP_BOT;
> > +import static com.performizeit.mjstack.monads.StepProps.KEEP_TOP;
>
> Why does the class MJStack need to know about all the step props ?
> Will look specificly at the code after pulling
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/lifey/mjstack/pull/14/files#r11096210>
> .
>