Skip to content

Commit

Permalink
Remove Checkstyle
Browse files Browse the repository at this point in the history
We haven't been checking the Checkstyle reports for a long time
and haven't been fixing any violations, so it's safe to say we
don't really need or care for using Checkstyle in Dropwizard.
  • Loading branch information
joschi committed Mar 12, 2019
1 parent 6940bde commit 437fb4d
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 120 deletions.
75 changes: 0 additions & 75 deletions checkstyle.xml

This file was deleted.

20 changes: 0 additions & 20 deletions dropwizard-example/pom.xml
Expand Up @@ -262,26 +262,6 @@
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
<version>3.0.0</version>
<configuration>
<configLocation>../checkstyle.xml</configLocation>
</configuration>
<executions>
<execution>
<id>checkstyle</id>
<phase>validate</phase>
<goals>
<goal>check</goal>
</goals>
<configuration>
<failOnViolation>true</failOnViolation>
</configuration>
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
Expand Down
25 changes: 0 additions & 25 deletions pom.xml
Expand Up @@ -547,11 +547,6 @@
<artifactId>maven-site-plugin</artifactId>
<version>3.7.1</version>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
<version>3.0.0</version>
</plugin>
<plugin>
<groupId>org.owasp</groupId>
<artifactId>dependency-check-maven</artifactId>
Expand Down Expand Up @@ -662,26 +657,6 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-project-info-reports-plugin</artifactId>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
<configuration>
<configLocation>checkstyle.xml</configLocation>
</configuration>
<executions>
<execution>
<id>checkstyle</id>
<phase>validate</phase>
<goals>
<goal>check</goal>
</goals>
<configuration>
<failOnViolation>true</failOnViolation>
</configuration>
</execution>
</executions>
</plugin>

<plugin>
<groupId>org.owasp</groupId>
<artifactId>dependency-check-maven</artifactId>
Expand Down

0 comments on commit 437fb4d

Please sign in to comment.