On Tue, 2018-09-04 at 13:25 -0400, Daniel Klco wrote:
> Robert,
> 
> Do you happen to know where that'd be? I checked the trunk POM[1] and
> the
> groovy config[2] didn't see any mention of Sonar.


According to our website ( 
https://sling.apache.org/documentation/development.html ) it used to be
at 
https://analysis.apache.org/dashboard/index/org.apache.sling:sling-builder
 .

But that's long gone.

> 
> Partially as an update, partially as documentation since I need to
> work on
> something else for a bit, I think this will just require updating the
> groovy job to add sonar as a publisher in the maven jobs[3].

Yes, it should be possible to 'just' add a sonar publisher. The only
note is that we should not do it for each commit, but rather every day,
if changes were applied.

On the other hand, the Sonar at Apache page tells us to talk to infra
for setting up Sonar analysis, see

  https://wiki.apache.org/general/SonarInstance

So I'd suggest starting there and seeing how they can accomodate our
large number of modules.

Thanks,

Robert

> 
> What I'm not sure about is that Apache Flex[4] (which has SonarQube
> working) has the SonarQube information provided in the POM. I *think*
> that
> should be provided automatically by the global Jenkins configuration.
> If
> that's not the case or if there are other Sling "global" settings
> we'd
> want, this may also require updating the parent POM.




> 
> -Dan
> 
> 1 - http://svn.apache.org/viewvc/sling/trunk/pom.xml?view=log
> 2 -
> 
http://svn.apache.org/viewvc/sling/trunk/tooling/jenkins/create_jobs.groovy?revision=1812253&view=markup
> 3 -
> 
https://jenkinsci.github.io/job-dsl-plugin/#path/mavenJob-publishers-sonar
> 4 - https://github.com/apache/flex-blazeds/blob/master/pom.xml
> 
> 
> On Tue, Sep 4, 2018 at 9:45 AM Robert Munteanu <[email protected]>
> wrote:
> 
> > On Tue, 2018-09-04 at 08:38 -0400, Daniel Klco wrote:
> > > Is that something others would be interested in having for our
> > > modules? If
> > > so, I'll pilot out getting this in place.
> > 
> > +1
> > 
> > Note that I think we have something setup for the old SVN reactor.
> > 
> > Robert
> > 
> > 


Reply via email to