Github user PascalSchumacher commented on a diff in the pull request:

    https://github.com/apache/commons-lang/pull/299#discussion_r147560886
  
    --- Diff: .travis.yml ---
    @@ -21,8 +21,19 @@ jdk:
       - oraclejdk8
       - oraclejdk9
     
    +cache:
    +  directories:
    +    - "$HOME/.m2/repository"
    +
    +before_cache:
    +  - rm -rf $HOME/.m2/repository/org/apache/commons/commons-lang3
    +
    +install:
    +  - mvn --version
    +
     script:
    -  - mvn
    +  - mvn -e -B
     
     after_success:
    -  - mvn clean cobertura:cobertura coveralls:report -Ptravis-cobertura
    +  - if [ $TRAVIS_JDK_VERSION == "openjdk7" ] || [ $TRAVIS_JDK_VERSION == 
"oraclejdk8" ]; then mvn -e -B clean cobertura:cobertura coveralls:report 
-Ptravis-cobertura; fi
    +  - if [ $TRAVIS_JDK_VERSION == "oraclejdk9" ]; then mvn -e -B clean 
cobertura:cobertura -Ptravis-cobertura; fi
    --- End diff --
    
    Nitpick: I think we can remove this line, because running `cobertura` 
without uploading to `coveralls` mostly just wastes cpu cycles.


---

Reply via email to