Hi Robert, It would be better to release that snapshot dependency in order to provide a way to easily test Sightly artifacts.
I'll take care of that this evening. Cheers, Radu On Monday, March 23, 2015, Robert Munteanu <[email protected]> wrote: > On Mon, 2015-03-23 at 10:50 -0400, Daniel Klco wrote: > > org.apache.sling.sightly.testing relies on a SNAPSHOT dependency: > > > > <dependency> > > <groupId>org.apache.sling</groupId> > > <artifactId>org.apache.sling.performance.base</artifactId> > > <version>0.0.1-SNAPSHOT</version> > > <scope>test</scope> > > </dependency> > > > > As this is a testing project, I'm not sure if this is a big deal or > > acceptable. Ideally, if we could make a release of this performance > > tools > > it would be good to apply performance tests across all of the > > scripting > > languages available in Sling. > > Good catch! I assumed the maven release plugin would be banning > SNAPSHOTs, but apparently this is not the case ... > > We can't deploy SNAPSHOT depedencies on Maven Central and we should not > as a general rule depend on SNAPSHOTs from releases, as the SNAPSHOTs > will at some point go away and make the released artifact invalid. > > So we either release the org.apache.sling.performance.base artifact and > depend on it or drop the org.apache.sling.sightly.testing artifact from > the release. > > Radu? > > Robert > > > -- Sent with clumsy fingers from a touchscreen
