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