Code review comment for lp:~beuno/loggerhead/yui3-0-0

Revision history for this message
Matt Nordhoff (mnordhoff) wrote :

I just played around with this a little, and everything worked (aside from the search bug, obviously).

But I know nothing about YUI, and not very much about JavaScript, so I'm not fit to review this.

I'm -0 on merging this before the search bug is fixed. I won't stop you, but I'd prefer the trunk remain as bug-free as possible, especially since the YUI upgrade doesn't really have an impact on users, and that bug does.

review: Abstain

« Back to merge proposal