Benjamin Bentmann wrote:
>> Please vote for the promotion of l10n plugin from the sandbox:
>
> +1, any plugin out of the sandbox is less hassle for uses to use it and
> provide feedback.
>
> Before the actual release, somebody could take care of these:
Thanks for pointing these out!
> Both mojos use j.l.String instead of j.i.File for file parameters and are
> therefore subject to relative path issues.
Fixed
> The status report has a user-accessible parameter taking model.Resource
> which is a further source of relative paths and needs manual resolving
> (like
> in MRESOURCES-32).
I'm not sure that this really needs to be accessible to the user. If it
doesn't need to, we can make it read-only. That would solve the problem,
right?
> It seems odd that the l10n:pseudo operates on "target/classes" (hence
> taking
> generated/filtered resources into account) while l10n:report by default
> only
> operates on the static resources in "src/main/resources" etc.
Not sure why it is like that.
> The plugin's own localization support should be completed
> (L10NStatusReport.java:559)
Fixed
> A sample report from the plugin about its own bundles would be a nice
> addition to its own site docs. I guess we could find some cute guys to
> add further bundles to make it more interesting ;-)
Report added. Now I'm waiting for translations ;-)
> Checkstyle errors could be fixed. Especially the report's renderBody()
> might
> benefit from refactoring out the analysis stuff from the actual report
> generation.
That is one big method! I don't dare chop that up.
> Benjamin
--
Dennis Lundberg
---------------------------------------------------------------------
To unsubscribe from this list, please visit:
http://xircles.codehaus.org/manage_email
