[GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...
Github user sesuncedu commented on the issue: https://github.com/apache/commons-compress/pull/39 Replaced by PR https://github.com/apache/commons-compress/pull/43 --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...
Github user sesuncedu commented on the issue: https://github.com/apache/commons-compress/pull/39 Closing because there's a new PR without the coveralls bait. Coveralls complains. The mvnw is not essential yet, but it's good to have 3.5.0 in place. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/39 [![Coverage Status](https://:/builds/12118668/badge)](https://:/builds/12118668) Coverage increased (+0.04%) to 84.766% when pulling **233536b0d1b1df8296fe254c59af5bed7e8c3e32 on sesuncedu:COMPRESS-413** into **4be9979b45ceadc50dc24607884d34613fead1f5 on apache:master**. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...
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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...
Github user sesuncedu commented on the issue: https://github.com/apache/commons-compress/pull/39 Randomly adding coverage tests for coveralls --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/39 [![Coverage Status](https://:/builds/12093740/badge)](https://:/builds/12093740) Coverage increased (+0.1%) to 84.828% when pulling **a434e5866df38e4b1f089c280f0b1a784e41a86d on sesuncedu:COMPRESS-413** into **4be9979b45ceadc50dc24607884d34613fead1f5 on apache:master**. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/39 [![Coverage Status](https://:/builds/12093695/badge)](https://:/builds/12093695) Coverage increased (+0.09%) to 84.815% when pulling **a434e5866df38e4b1f089c280f0b1a784e41a86d on sesuncedu:COMPRESS-413** into **4be9979b45ceadc50dc24607884d34613fead1f5 on apache:master**. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/39 [![Coverage Status](https://:/builds/12090926/badge)](https://:/builds/12090926) Coverage decreased (-0.008%) to 84.72% when pulling **7b4ecc9e7c9d2fb32ddfe4f92791aacf9f17d901 on sesuncedu:COMPRESS-413** into **4be9979b45ceadc50dc24607884d34613fead1f5 on apache:master**. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/39 [![Coverage Status](https://:/builds/12090041/badge)](https://:/builds/12090041) Coverage remained the same at 84.728% when pulling **95974fc671661244c83fdead2231f8367a67d2ba on sesuncedu:COMPRESS-413** into **4be9979b45ceadc50dc24607884d34613fead1f5 on apache:master**. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---