Review: Approve code, unittests
Hi Seif,
thanks for working on this. It's looking good and all tests are working fine.
Feel free to do the merge.
Markus
--
https://code.launchpad.net/~zeitgeist/zeitgeist/fix-655164/+merge/38077
Your team Zeitgeist Framework Team is subscribed to branch
lp:~zeit
On Mon, Oct 11, 2010 at 4:20 PM, Markus Korn wrote:
> I did not have time yet to check if the tests make sense, but let me give
> you a few comments first:
>
>
> > === modified file '_zeitgeist/engine/main.py'
> > --- _zeitgeist/engine/main.py 2010-09-29 08:39:32 +
> > +++ _zeitgeist/engine
I did not have time yet to check if the tests make sense, but let me give you a
few comments first:
> === modified file '_zeitgeist/engine/main.py'
> --- _zeitgeist/engine/main.py 2010-09-29 08:39:32 +
> +++ _zeitgeist/engine/main.py 2010-10-10 14:57:44 +
> @@ -370,7 +370,15 @@
>
3 matches
Mail list logo