[Dextrose] ticket #2318 display free journal space in volume toolbar
Aleksey Lim
alsroot at member.fsf.org
Sat Oct 16 15:57:04 EDT 2010
On Sun, Oct 17, 2010 at 12:58:27AM +0530, Ishan Bansal wrote:
> Hi Aleksey
>
> I had submitted the patch for the http://bugs.sugarlabs.org/ticket/2318
>
> You can check the patch at
> http://www.mail-archive.com/sugar-devel@lists.sugarlabs.org/msg16721.html
>
> Wish if you could review it and provide me feedback on any improvement
> required.
You need to check your patch (only your patch, the rest of code does not
conform all checks) w/ pylint/pep8 (eg using sugar-lint[1]).
> + label = 'Journal'
string needs to be kept gettextized ie _('Journal')
[1] http://wiki.sugarlabs.org/go/Activity_Team/Sugar_Lint
--
Aleksey
More information about the Dextrose
mailing list