[GitHub] commons-compress issue #39: COMPRESS-413: Performance improvements for Travi...

2017-06-24 Thread sesuncedu
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...

2017-06-24 Thread sesuncedu
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...

2017-06-24 Thread coveralls
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...

2017-06-24 Thread bodewig
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...

2017-06-22 Thread sesuncedu
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...

2017-06-22 Thread coveralls
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...

2017-06-22 Thread coveralls
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...

2017-06-22 Thread coveralls
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...

2017-06-22 Thread coveralls
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.
---