Tell sonar to not check Messages classes

NLS Message classes use a non Java compliant formating of field names. However this is the common practice to do so. Therefore we are are suppressing Sonars rule squid:S3008 - Static non-final field names should comply with a naming convention. to not disturb us with this noise.
azoitl · Dec 17, 2023 · 63a0b60 · 63a0b60
1 parent 98a6240
commit 63a0b60
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 0 deletions.
diff --git a/org.eclipse.gef.examples.logic/src/org/eclipse/gef/examples/logicdesigner/LogicMessages.java b/org.eclipse.gef.examples.logic/src/org/eclipse/gef/examples/logicdesigner/LogicMessages.java
@@ -14,6 +14,7 @@
 
 import org.eclipse.osgi.util.NLS;
 
+@SuppressWarnings("squid:S3008") // tell sonar the java naming convention does not make sense for this class
 public class LogicMessages extends NLS {
 
 	public static String AddCommand_Label;

diff --git a/...lipse.gef.examples.ui.pde/src/org/eclipse/gef/examples/ui/pde/internal/l10n/Messages.java b/...lipse.gef.examples.ui.pde/src/org/eclipse/gef/examples/ui/pde/internal/l10n/Messages.java
@@ -14,6 +14,7 @@
 
 import org.eclipse.osgi.util.NLS;
 
+@SuppressWarnings("squid:S3008") // tell sonar the java naming convention does not make sense for this class
 public final class Messages extends NLS {
 
 	private static final String BUNDLE_NAME = "org.eclipse.gef.examples.ui.pde.internal.l10n.Messages";//$NON-NLS-1$

diff --git a/org.eclipse.gef/src/org/eclipse/gef/internal/GEFMessages.java b/org.eclipse.gef/src/org/eclipse/gef/internal/GEFMessages.java
@@ -20,6 +20,7 @@
  * @author hudsonr
  * @since 2.0
  */
+@SuppressWarnings("squid:S3008") // tell sonar the java naming convention does not make sense for this class
 public class GEFMessages extends NLS {
 
 	public static String AlignBottomAction_Label;

diff --git a/org.eclipse.gef/src/org/eclipse/gef/ui/palette/PaletteMessages.java b/org.eclipse.gef/src/org/eclipse/gef/ui/palette/PaletteMessages.java
@@ -17,6 +17,7 @@
 /**
  * @author Pratik Shah
  */
+@SuppressWarnings("squid:S3008") // tell sonar the java naming convention does not make sense for this class
 public class PaletteMessages extends NLS {
 
 	/**
-Original file line number
+Diff line change
@@ Expand Up / @@ -17,6 +17,7 @@ @@
     /**
      * @author Pratik Shah
      */
+    @SuppressWarnings("squid:S3008") // tell sonar the java naming convention does not make sense for this class
     public class PaletteMessages extends NLS {
     	/**
@@ Expand Down @@