Review of lightread

Didier Roche didrocks at ubuntu.com
Mon Aug 6 13:23:05 UTC 2012


Hey guys,

As dpm asked me, I did a review of lightread source code and packaging 
themselves.

The code is cleaned, using sass and some popular javascript libraries, 
clearly showing the guy know what he is doing. I just removed an 
harmless .DS_Store which isn't useful in this context, as well as a 
readabilty.js unsued file. Even if it's unused, the goal of this file is 
to use a third-party service to prettify the blog entry. There is a 
token and so on. However, this one isn't even called, so I felt better 
to completely remove it (which is just about removing that file right now).

https://code.launchpad.net/~didrocks/ubuntu-app-reviews/lightread

The packaging itself works from /opt and comply the standards of the 
ARB. So a definitive +1 from me.

Cheers,
Didier



More information about the App-review-board mailing list