directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Emmanuel Lécharny <elecha...@gmail.com>
Subject Re: [Fortress] Sonar...
Date Thu, 21 May 2015 12:09:32 GMT
Le 21/05/15 13:32, Shawn McKinney a écrit :
>> On May 20, 2015, at 8:52 AM, Emmanuel Lécharny <elecharny@gmail.com> wrote:
>>
>> I would say : fix whatever seems simple to fix. Low hanging fruits
>> first, after of course critical and blocking issues.
>>
>> I have asked some creds to log on the tool to be able to tune some of
>> the rules.
>>
>> We can discuss specific rules that seems a bit too strict, if needed.
>>
>>
>> The value I see in such a tool is mainly about reviewing the code, which
>> allows us to find some real and hidden issues, like the one on roles...
> Most of the dependency cycles are fixed by moving the core's entities to separate package.
 Easy to do, but breaks backward compatibility with existing clients.

Don't do that. This warning is not really useful, considering that all
the classes are in the same project.

>From a OSGi PoV, this is not a problem either.


Mime
View raw message