[Review] Lionss: Website for string search
Riccardo Magliocchetti
riccardo.magliocchetti at gmail.com
Mon Jul 7 00:55:02 PDT 2014
Hello Mat,
Il 06/07/2014 02:05, Mat M ha scritto:
> Hello
>
> An old thread started in l10n list about improving translations. This
> lead to a cross-post to the dev ML [0], where I said I will implement a
> web interface to search for GUI strings, find their usage and opengrok
> them.
Nice!
> Some time later, I eventually did (apart of) the job [1], which is
> reachable through [2].
> With the kind comment of Michael M in [3], I decided to push to gerrit [4].
What about packaging it as a python application so that it is
installable with pip?
> Next steps are:
> * Review :)
From a quick look you have some whitespace issues highlighted by
gerrit, also classes should start with a Capital letter.
In config.py why not being explicit creating gg_settings?
gg_settings = [dict(pattern_prefix = '<property name="label"
translatable="yes">')]
also why not moving each variable declaration on its own line?
Haven't looked at the real code yet
Commenting here because am totally failing at understanding how to
promote my draft comments on gerrit :|
thanks,
riccardo
More information about the LibreOffice
mailing list