From 3683769be152e4f03239011b85e81a9e37938241 Mon Sep 17 00:00:00 2001
From: Artur <artur@vaadin.com>
Date: Tue, 3 Jan 2017 10:25:14 +0200
Subject: [PATCH] Make it easy to run Sonar on the project (#8118)

---
 compatibility-client-compiled/pom.xml | 2 ++
 compatibility-client/pom.xml          | 5 ++++-
 compatibility-server/pom.xml          | 4 ++++
 compatibility-shared/pom.xml          | 5 +++++
 compatibility-themes/pom.xml          | 5 +++++
 pom.xml                               | 6 ++++++
 test/pom.xml                          | 3 +++
 uitest-common/pom.xml                 | 2 ++
 uitest/pom.xml                        | 2 ++
 9 files changed, 33 insertions(+), 1 deletion(-)

diff --git a/compatibility-client-compiled/pom.xml b/compatibility-client-compiled/pom.xml
index 298f402bf6..4bdf8c1fd6 100644
--- a/compatibility-client-compiled/pom.xml
+++ b/compatibility-client-compiled/pom.xml
@@ -15,6 +15,8 @@
         <widgetset.name>com.vaadin.v7.Vaadin7WidgetSet</widgetset.name>
         <widgetset.style>OBF</widgetset.style>
         <widgetset.local.workers>6</widgetset.local.workers>
+        <!-- These files are directly from Vaadin 7 -->
+        <sonar.skip>true</sonar.skip>
     </properties>
 
     <url>https://vaadin.com/</url>
diff --git a/compatibility-client/pom.xml b/compatibility-client/pom.xml
index b5bdf183a9..898326526c 100644
--- a/compatibility-client/pom.xml
+++ b/compatibility-client/pom.xml
@@ -13,7 +13,10 @@
 
     <url>https://vaadin.com/</url>
     <description>Vaadin 7 compatibility package for Vaadin 8</description>
-
+    <properties>
+        <!-- These files are directly from Vaadin 7 -->
+        <sonar.skip>true</sonar.skip>
+    </properties>
     <dependencies>
         <!-- Project modules -->
         <dependency>
diff --git a/compatibility-server/pom.xml b/compatibility-server/pom.xml
index 6902c52536..500aa0db2f 100644
--- a/compatibility-server/pom.xml
+++ b/compatibility-server/pom.xml
@@ -14,6 +14,10 @@
     <url>https://vaadin.com/</url>
     <description>Vaadin 7 compatibility package for Vaadin 8</description>
 
+    <properties>
+        <!-- These files are directly from Vaadin 7 -->
+        <sonar.skip>true</sonar.skip>
+    </properties>
     <dependencies>
         <!-- Project modules -->
         <dependency>
diff --git a/compatibility-shared/pom.xml b/compatibility-shared/pom.xml
index 30886a1ac1..d702702748 100644
--- a/compatibility-shared/pom.xml
+++ b/compatibility-shared/pom.xml
@@ -14,6 +14,11 @@
     <url>https://vaadin.com/</url>
     <description>Vaadin 7 compatibility package for Vaadin 8</description>
 
+    <properties>
+        <!-- These files are directly from Vaadin 7 -->
+        <sonar.skip>true</sonar.skip>
+    </properties>
+
     <dependencies>
         <!-- Project modules -->
         <dependency>
diff --git a/compatibility-themes/pom.xml b/compatibility-themes/pom.xml
index ed47a260e0..d373cee9ac 100644
--- a/compatibility-themes/pom.xml
+++ b/compatibility-themes/pom.xml
@@ -14,6 +14,11 @@
     <url>https://vaadin.com/</url>
     <description>Vaadin compatibility themes</description>
 
+    <properties>
+        <!-- These files are directly from Vaadin 7 -->
+        <sonar.skip>true</sonar.skip>
+    </properties>
+
     <dependencies>
         <!-- Maven plugin scans for vaadin-server... -->
         <dependency>
diff --git a/pom.xml b/pom.xml
index 7d14fc9c75..9a52e31c3f 100644
--- a/pom.xml
+++ b/pom.xml
@@ -47,6 +47,12 @@
         
         <jetty.version>9.3.7.v20160115</jetty.version>
         <phantomjs.version>2.1.1</phantomjs.version>
+
+        <!-- Sonar properties -->
+        <sonar.java.source>8</sonar.java.source>
+        <sonar.analysis.mode>preview</sonar.analysis.mode>
+        <sonar.issuesReport.console.enable>true</sonar.issuesReport.console.enable>
+        <sonar.issuesReport.html.enable>true</sonar.issuesReport.html.enable>
     </properties>
 
     <!-- TODO: remove this after maven plugin has been released -->
diff --git a/test/pom.xml b/test/pom.xml
index d5502dc6e9..e13f51b497 100644
--- a/test/pom.xml
+++ b/test/pom.xml
@@ -15,6 +15,9 @@
         <phantomjs.version>2.1.1</phantomjs.version>
         <vaadin.version>8.0-SNAPSHOT</vaadin.version>
         <failOnMissingWebXml>false</failOnMissingWebXml>
+        <!-- Don't care about coding style for tests -->
+        <sonar.skip>true</sonar.skip>
+    </properties>
     </properties>
 
     <repositories>
diff --git a/uitest-common/pom.xml b/uitest-common/pom.xml
index 28ae930e24..f87ece9f3a 100644
--- a/uitest-common/pom.xml
+++ b/uitest-common/pom.xml
@@ -12,6 +12,8 @@
     <packaging>jar</packaging>
     <properties>
         <skip.uitest.deployment>true</skip.uitest.deployment>
+        <!-- Don't care about coding style for tests -->
+        <sonar.skip>true</sonar.skip>
     </properties>
 
     <url>https://vaadin.com/</url>
diff --git a/uitest/pom.xml b/uitest/pom.xml
index 8f9fb53758..84eeb9566d 100644
--- a/uitest/pom.xml
+++ b/uitest/pom.xml
@@ -12,6 +12,8 @@
     <packaging>war</packaging>
     <properties>
         <skip.uitest.deployment>true</skip.uitest.deployment>
+        <!-- Don't care about coding style for tests -->
+        <sonar.skip>true</sonar.skip>
     </properties>
 
     <url>https://vaadin.com/</url>
-- 
2.39.5