commons-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bodewig <...@git.apache.org>
Subject [GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...
Date Sat, 24 Jun 2017 15:30:56 GMT
Github user bodewig commented on the issue:

    https://github.com/apache/commons-compress/pull/39
  
    Thanks Simon.
    
    TBH I'm not too sure why we've got Travis builds at all, probably so we can run Coveralls.
I'll leave this request open for those more interested in the Travis CI setup. I'll rather
invest some time in adding a Jenkins matrix build for the JDKs we want to support instead.
    
    Commons Compress aims to be compatible with Java7, no matter how old or deprecated, but
we already cover that with Jenkins - the pull request builder over there uses Oracle JDK 7,
so we've got that covered independent of Travis CI. We do run SonarQube for Compress already.
    
    Personally I dislike `mvnw` scripts, but this is probably a matter of taste. I won't veto
its introduction.
    
    Please strip the changes to `AsiExtraField` from this PR, they are completely unrelated.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message