refactor: Common static analysis issues

src/main/java/io/moderne/organizations/OrganizationRepository.java

@@ -9,7 +9,7 @@ public OrganizationRepository(String path, String origin, String branch) {
         this.origin = origin;
         this.path = path;
         this.branch = branch;
-        if (origin.equals("*") && path.equals("*") && branch.equals("*")) {
+        if ("*".equals(origin) && "*".equals(path) && "*".equals(branch)) {
             throw new IllegalArgumentException("Cannot have a repository with origin=* and branch=* and path=* as this would match all repositories");
         }
     }

src/main/java/io/moderne/organizations/types/Dashboard.java

@@ -65,8 +65,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         Dashboard that = (Dashboard) o;
         return java.util.Objects.equals(upgradesAndMigrations, that.upgradesAndMigrations) &&
                             java.util.Objects.equals(visualizations, that.visualizations) &&

src/main/java/io/moderne/organizations/types/DashboardRecipe.java

@@ -41,8 +41,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         DashboardRecipe that = (DashboardRecipe) o;
         return java.util.Objects.equals(id, that.id) &&
                             java.util.Objects.equals(options, that.options);

src/main/java/io/moderne/organizations/types/DashboardVisualization.java

@@ -52,8 +52,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         DashboardVisualization that = (DashboardVisualization) o;
         return java.util.Objects.equals(recipe, that.recipe) &&
                             java.util.Objects.equals(id, that.id) &&

src/main/java/io/moderne/organizations/types/DevCenter.java

@@ -71,8 +71,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         DevCenter that = (DevCenter) o;
         return java.util.Objects.equals(upgradesAndMigrations, that.upgradesAndMigrations) &&
                             java.util.Objects.equals(visualizations, that.visualizations) &&

src/main/java/io/moderne/organizations/types/DevCenterRecipe.java

@@ -41,8 +41,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         DevCenterRecipe that = (DevCenterRecipe) o;
         return java.util.Objects.equals(recipeId, that.recipeId) &&
                             java.util.Objects.equals(options, that.options);

src/main/java/io/moderne/organizations/types/DevCenterRecipeCard.java

@@ -52,8 +52,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         DevCenterRecipeCard that = (DevCenterRecipeCard) o;
         return java.util.Objects.equals(title, that.title) &&
                             java.util.Objects.equals(measures, that.measures) &&

src/main/java/io/moderne/organizations/types/DevCenterVisualizationCard.java

@@ -41,8 +41,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         DevCenterVisualizationCard that = (DevCenterVisualizationCard) o;
         return java.util.Objects.equals(visualizationId, that.visualizationId) &&
                             java.util.Objects.equals(visualizationOptions, that.visualizationOptions);

src/main/java/io/moderne/organizations/types/Measure.java

@@ -40,8 +40,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         Measure that = (Measure) o;
         return java.util.Objects.equals(name, that.name) &&
                             java.util.Objects.equals(recipe, that.recipe);

src/main/java/io/moderne/organizations/types/Option.java

@@ -51,8 +51,12 @@ public String toString() {
 
   @Override
   public boolean equals(java.lang.Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         Option that = (Option) o;
         return java.util.Objects.equals(name, that.name) &&
                             java.util.Objects.equals(value, that.value);

src/main/java/io/moderne/organizations/types/Organization.java

@@ -81,8 +81,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         Organization that = (Organization) o;
         return java.util.Objects.equals(id, that.id) &&
                             java.util.Objects.equals(name, that.name) &&

src/main/java/io/moderne/organizations/types/RepositoryInput.java

@@ -51,8 +51,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         RepositoryInput that = (RepositoryInput) o;
         return java.util.Objects.equals(path, that.path) &&
                             java.util.Objects.equals(origin, that.origin) &&

src/main/java/io/moderne/organizations/types/User.java

@@ -29,8 +29,12 @@ public String toString() {
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+    if (this == o) {
+      return true;
+    }
+    if (o == null || getClass() != o.getClass()) {
+      return false;
+    }
         User that = (User) o;
         return java.util.Objects.equals(email, that.email);
   }