Le 7 mars 2013 08:17, "Vincent Massol" <[email protected]> a écrit :
>
>
> On Mar 6, 2013, at 7:19 PM, Sergiu Dumitriu <[email protected]> wrote:
>
> > On 03/06/2013 12:54 PM, Jeremie BOUSQUET wrote:
> >> Hello guys,
> >>
> >> So it's done, and I've published both versions 1.0.1 (from the original
> >> from xwiki.org) and 1.0.2 (wich fixes excessive empty lines inserted)
both
> >> to github and to nexus staging.
> >>
> >> If someone could kindly promote those artifacts from Nexus staging,
that
> >> would be nice :-)
> >>
> >> groupId: org.xwiki.contrib
> >> artifactId: xwiki-macro-tabs
> >> versions: 1.0.1 and 1.0.2
> >
> > Very nice POM!
>
> indeed!
>
> Only thing I'd comment is to have consistent # of chars for indentation:
in some places you use 20 chars for indentation and at other places 6, and
at others 1 and at others 2 :)
>
> We use 2 chars everywhere so I'd suggest to use 2 for not having too long
lines.

You're right, I forgot to beautify it ... I'll fix that for next versions.


Thanks,

Jeremie

>
> Thanks
> -Vincent
>
> > I tested the XAR, seems to be working fine. Repo released:
> >
http://nexus.xwiki.org/nexus/content/repositories/releases/org/xwiki/contrib/xwiki-macro-tabs/
> >
> >> Note: as I put "<xwiki.extension.name>Tabs Macro</xwiki.extension.name>"
in
> >> the pom, and page name on xwiki.org is "Tabs Macro", I believe that if
I
> >> "import" from Extension Repository, it should update the extension page
> >> accordingly, is this right ? Because the extension id is different now
of
> >> course...
> _______________________________________________
> users mailing list
> [email protected]
> http://lists.xwiki.org/mailman/listinfo/users
_______________________________________________
users mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/users

Reply via email to