Code review comment for lp://staging/~doanac/ubuntu-ci-services-itself/json-status-styling

Revision history for this message
Andy Doan (doanac) wrote :

On 03/06/2014 09:39 AM, Joe Talbott wrote:
> L55 remove 2013 I think

I'll fix this.

> L254 s/!=/!==/

I think you meant L242. Regardless:

<https://code.launchpad.net/~doanac/ubuntu-ci-services-itself/json-datatables/+merge/209506>

removes all the ugly stuff, so I'd rather just merge with this ugliness
for now if you don't mind?

>
> L260 not needed I think

ditto for this.

« Back to merge proposal