diff options
| author | Evgeny Mandrikov <mandrikov@gmail.com> | 2012-07-05 16:40:24 +0600 |
|---|---|---|
| committer | Evgeny Mandrikov <mandrikov@gmail.com> | 2012-07-05 18:13:33 +0600 |
| commit | e394a262f87201ea9c646495e6c06f793447caf9 (patch) | |
| tree | 9e54b1c2df4dab7c64a25b91e55567a8d495df9c /plugins/sonar-l10n-en-plugin | |
| parent | d963ab5619aa75d28b7cc981f0615c67f73d1fd5 (diff) | |
| download | sonarqube-e394a262f87201ea9c646495e6c06f793447caf9.tar.gz sonarqube-e394a262f87201ea9c646495e6c06f793447caf9.zip | |
SONAR-2333 Add new rule to report about cycles between packages
Diffstat (limited to 'plugins/sonar-l10n-en-plugin')
2 files changed, 9 insertions, 0 deletions
diff --git a/plugins/sonar-l10n-en-plugin/src/main/resources/org/sonar/l10n/squidjava.properties b/plugins/sonar-l10n-en-plugin/src/main/resources/org/sonar/l10n/squidjava.properties index c178b1b078c..d74bc3c0bf5 100644 --- a/plugins/sonar-l10n-en-plugin/src/main/resources/org/sonar/l10n/squidjava.properties +++ b/plugins/sonar-l10n-en-plugin/src/main/resources/org/sonar/l10n/squidjava.properties @@ -29,3 +29,5 @@ rule.squid.UnusedPrivateMethod.name=Unused private method rule.squid.UnusedProtectedMethod.name=Unused protected method rule.squid.CommentedOutCodeLine.name=Avoid commented-out lines of code + +rule.squid.CycleBetweenPackages.name=Avoid cycle between java packages diff --git a/plugins/sonar-l10n-en-plugin/src/main/resources/org/sonar/l10n/squidjava/rules/squid/CycleBetweenPackages.html b/plugins/sonar-l10n-en-plugin/src/main/resources/org/sonar/l10n/squidjava/rules/squid/CycleBetweenPackages.html new file mode 100644 index 00000000000..6bab29f493b --- /dev/null +++ b/plugins/sonar-l10n-en-plugin/src/main/resources/org/sonar/l10n/squidjava/rules/squid/CycleBetweenPackages.html @@ -0,0 +1,7 @@ +<p> +When several packages are involved in a cycle (package A > package B > package C > package A where ">" means "depends upon"), +that means that those packages are highly coupled and that there is no way to reuse/extract one of those packages without importing all the other packages. +Such cycle could quickly increase the effort required to maintain an application and to embrace business change. +Sonar not only detect cycles between packages but also determines what is the minimum effort to break those cycles. +This rule log a violation on each source file having an outgoing dependency to be but in order to break a cycle. +</p> |