Code review comment for lp://staging/~gary/launchpad/move-events-to-filters

Revision history for this message
Graham Binns (gmb) wrote :

Hi Gary,

So, I'm broadly-speaking +1 on this branch landing (note that I'm on a fairly narrow bandwidth at the moment, so I've gone straight for understanding the code and not worried too much about formatting issues). However, I don't feel that I have enough knowledge about the way that filters work at the moment to offer a definite r=me. I'd suggest you ask Gavin to cast a weather eye over it too, since the filters were at least in part his work.

Anyway, approval from me. Nice work; I don't think I can see a way to break it up without causing some heartache, so don't worry about the branch size.

review: Approve (code)

« Back to merge proposal