Adapt_Linter, Remove Travis CI, Update Tests and Readme (#46)

* Try to disable linting of changes

* Remove Slack reporting

* Try to reactivate escape comments

* Try to reactivate escape comments

* Try to reactivate escape comments

* Deactivate Line Length test

* Update sun_checks.xml

* Update README.md

* Remove Travis Traces

Author: cdr-chakotay

.github/linters/sun_checks.xml

@@ -71,10 +71,10 @@
   <!-- Checks for Size Violations.                    -->
   <!-- See https://checkstyle.org/config_sizes.html -->
   <module name="FileLength"/>
-  <module name="LineLength">
+  <!--module name="LineLength">
     <property name="fileExtensions" value="java"/>
     <property name="max" value="120"/>
-  </module>
+  </module>                                       -->
 
   <!-- Checks for whitespace                               -->
   <!-- See https://checkstyle.org/config_whitespace.html -->
@@ -205,7 +205,6 @@
       <property name="optional" value="true"/>
     </module>
     <module name="SuppressionCommentFilter"/>
-
-  </module>
+    </module>
 
 </module>

.github/workflows/lintChanges.yml

@@ -21,6 +21,7 @@ jobs:
         uses: github/super-linter@v3
         env:
           VALIDATE_ALL_CODEBASE: true # lint all files
+          FILTER_REGEX_EXCLUDE: /src/test/*
           VALIDATE_JAVA: true  # only lint Java files
           DEFAULT_BRANCH: master
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Enables better overview of runs

.github/workflows/tests.yml

@@ -33,16 +33,4 @@ jobs:
       - name: Build with Gradle
         run: ./gradlew assemble
       - name: Run Tests
-        run: ./gradlew check
-
-    slack-on-failure:
-      needs: [Java]
-      if: failure() && github.ref == 'refs/heads/master'
-      runs-on: ubuntu-latest
-
-      steps:
-        - uses: 8398a7/action-slack@v3
-          with:
-            status: failure
-          env:
-            SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
+        run: ./gradlew check

.travis.yml

@@ -1,4 +1,4 @@
-# tus-java-client [![Build Status](https://travis-ci.org/tus/tus-java-client.svg?branch=master)](https://travis-ci.org/tus/tus-java-client)
+# tus-java-client [![Tests](https://github.com/tus/tus-java-client/actions/workflows/tests.yml/badge.svg?branch=master)](https://github.com/tus/tus-java-client/actions/workflows/tests.yml)
 
 > **tus** is a protocol based on HTTP for *resumable file uploads*. Resumable
 > means that an upload can be interrupted at any moment and can be resumed without

README.md

@@ -319,8 +319,9 @@ public TusUploader resumeOrCreateUpload(@NotNull TusUpload upload) throws Protoc
     public void prepareConnection(@NotNull HttpURLConnection connection) {
         // Only follow redirects, if the POST methods is preserved. If http.strictPostRedirect is
         // disabled, a POST request will be transformed into a GET request which is not wanted by us.
+
         // CHECKSTYLE:OFF
-        // Necessary because of length of the link
+        // LineLength - Necessary because of length of the link
         // See:https://github.com/openjdk/jdk/blob/jdk7-b43/jdk/src/share/classes/sun/net/www/protocol/http/HttpURLConnection.java#L2020-L2035
         // CHECKSTYLE:ON
         connection.setInstanceFollowRedirects(Boolean.getBoolean("http.strictPostRedirect"));