Update EditorConfig and fix trailing whitespace and end of files

.editorconfig

@@ -1,10 +1,11 @@
 root = true
 
 [*]
-indent_style = space
-indent_size = 4
-end_of_line = lf
 charset = utf-8
-trim_trailing_whitespace = true
+end_of_line = lf
+indent_size = 4
+indent_style = space
 insert_final_newline = true
-max_line_length = off
+max_line_length = 120
+tab_width = 4
+trim_trailing_whitespace = true

README.md

@@ -109,7 +109,7 @@ bacon pig <phase> ...
 ```
 ## Configuration
 
-The application is configured via the `build-config.yaml` file. 
+The application is configured via the `build-config.yaml` file.
 
 A PiG `build-config.yaml` looks like this: https://github.com/project-ncl/bacon/blob/master/example-pig-config.yaml
 

auth/pom.xml

@@ -65,4 +65,4 @@
             <artifactId>jackson-datatype-jsr310</artifactId>
         </dependency>
     </dependencies>
-</project>
+</project>

common/src/main/java/org/jboss/pnc/bacon/common/cli/AbstractNotImplementedCommand.java

@@ -46,4 +46,4 @@ public CommandResult execute(CommandInvocation commandInvocation) throws Command
         log.error("Not implemented yet!");
         return CommandResult.FAILURE;
     }
-}
+}

integration-tests/pom.xml

@@ -27,13 +27,13 @@
             <groupId>org.jboss.pnc.bacon</groupId>
             <artifactId>common</artifactId>
         </dependency>
-        
+
         <!-- PNC deps -->
         <dependency>
             <groupId>org.jboss.pnc</groupId>
             <artifactId>dto</artifactId>
         </dependency>
-        
+
         <!-- Object mapper -->
         <dependency>
             <groupId>com.fasterxml.jackson.core</groupId>
@@ -47,7 +47,7 @@
             <groupId>com.fasterxml.jackson.datatype</groupId>
             <artifactId>jackson-datatype-jsr310</artifactId>
         </dependency>
-        
+
         <!-- Test framework -->
         <dependency>
             <groupId>org.junit.jupiter</groupId>

pig/src/main/java/org/jboss/pnc/bacon/pig/impl/common/DeliverableManager.java

@@ -38,13 +38,13 @@
  * The execution is controlled by a Groovy snippet. The manager is injected to the script under 'gen'.
  * </p>
  * E.g. to download an artifact matching *license.zip from a build named WildFly-Swarm, one would use:
- * 
+ *
  * <pre>
  *     gen.downloadFrom 'WildFly-Swarm' matching '.*license\\.zip'
  * </pre>
  * <p>
  * If another method, <code>generate()</code> is added to a custom manager, it can be used as follows:
- * 
+ *
  * <pre>
  * gen.generate()
  * </pre>

pig/src/main/java/org/jboss/pnc/bacon/pig/impl/config/Output.java

@@ -28,4 +28,4 @@
 public class Output {
     private String releaseFile;
     private String releaseDir;
-}
+}

pig/src/main/java/org/jboss/pnc/bacon/pig/impl/config/ProductConfig.java

@@ -34,4 +34,4 @@ public String prefix() {
         // noinspection StringToUpperCaseOrToLowerCaseWithoutLocale
         return getName().toLowerCase().replaceAll(" ", "-");
     }
-}
+}

pig/src/main/java/org/jboss/pnc/bacon/pig/impl/utils/MilestoneNumberFinder.java

@@ -29,7 +29,7 @@
 
 /**
  * TODO: resurrect it
- * 
+ *
  * @author Michal Szynkiewicz, [email protected] <br>
  *         Date: 3/5/18
  */

pig/src/main/resources/ARTIFACT_LIST.ftl

@@ -1 +1 @@
-${repositoryContents}
+${repositoryContents}

pig/src/main/resources/DUPLICATE_ARTIFACT_LIST.ftl

@@ -1 +1 @@
-${duplicates}
+${duplicates}

pig/src/main/resources/REPOSITORY_CONTENTS.ftl

@@ -1 +1 @@
-${repositoryContents}
+${repositoryContents}

pig/src/main/resources/REPOSITORY_COORDINATES.ftl

@@ -1,4 +1,4 @@
 <#list builds as build>
 ${build.name}_SCM_URL=${build.internalScmUrl}
 ${build.name}_SCM_REVISION=${build.scmRevision}
-</#list>
+</#list>

pig/src/main/resources/license-generator.properties

@@ -1,4 +1,4 @@
 # to run with PNC stage, you need to set the `INDY_REPO_URL` environment variable to http://indy-stage.psi.redhat.com/
 repository.names=${names}
 repository.urls=${urls}
-${licenseServiceUrl}
+${licenseServiceUrl}

pig/src/main/resources/repository-example-settings.xml

@@ -51,4 +51,4 @@
     <!-- Optionally, make the repository active by default -->
     <activeProfile>repository</activeProfile>
   </activeProfiles>
-</settings>
+</settings>

pig/src/test/java/org/jboss/pnc/bacon/pig/impl/documents/sharedcontent/da/DADaoTest.java

@@ -75,4 +75,4 @@ public void shouldLookupJunity456() {
         assertThat(dependency.getState()).isEqualTo(DependencyState.NO_MATCH);
     }
 
-}
+}

pnc/src/test/java/org/jboss/pnc/bacon/pnc/ProductVersionCliTest.java

@@ -19,4 +19,4 @@ void validateProductVersionTest() {
         assertThat(ProductVersionCli.validateProductVersion("42")).isFalse();
         assertThat(ProductVersionCli.validateProductVersion("42.1a")).isFalse();
     }
-}
+}

test-common/pom.xml

@@ -1,20 +1,19 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
-  <parent>
-    <artifactId>parent</artifactId>
-    <groupId>org.jboss.pnc.bacon</groupId>
-    <version>2.0.2-SNAPSHOT</version>
-  </parent>
-  <modelVersion>4.0.0</modelVersion>
+    <parent>
+        <artifactId>parent</artifactId>
+        <groupId>org.jboss.pnc.bacon</groupId>
+        <version>2.0.2-SNAPSHOT</version>
+    </parent>
+    <modelVersion>4.0.0</modelVersion>
 
-  <artifactId>test-common</artifactId>
+    <artifactId>test-common</artifactId>
 
-  <dependencies>
-    <dependency>
-      <groupId>org.junit.jupiter</groupId>
-      <artifactId>junit-jupiter-api</artifactId>
-      <scope>compile</scope>
-    </dependency>
-  </dependencies>
-
-</project>
+    <dependencies>
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-api</artifactId>
+            <scope>compile</scope>
+        </dependency>
+    </dependencies>
+</project>