[GitHub] commons-compress issue #70: Use Java 7 Files.createTempDirectory instead of ...

2018-10-03 Thread bodewig 
Github user bodewig commented on the issue:

https://github.com/apache/commons-compress/pull/70
  
I've just commited a minimal implementation of the suggested change, many 
thanks.


---

[GitHub] commons-compress issue #70: Use Java 7 Files.createTempDirectory instead of ...

2018-09-28 Thread bodewig 
Github user bodewig commented on the issue:

https://github.com/apache/commons-compress/pull/70
  
Many thanks. I agree this is cleaner, the existing method is a leftover of 
the time before we made Java7 our baseline.

While I don't think we consider the tests a public API of ours, the change 
would be smaller and keep perfect backwards compatibility if you "just" changed 
the implementation of `mkdir`.


---

[GitHub] commons-compress issue #70: Use Java 7 Files.createTempDirectory instead of ...

2018-09-28 Thread coveralls 
Github user coveralls commented on the issue:

https://github.com/apache/commons-compress/pull/70
  

[![Coverage 
Status](https://coveralls.io/builds/19243727/badge)](https://coveralls.io/builds/19243727)

Coverage remained the same at 86.781% when pulling 
**bb6890f0fe9bc51a9e493d9ee9ab02a04a155ac5 on DaGeRe:master** into 
**ac3d73153725fa1bce721055c4ac61423570d34c on apache:master**.



---

[GitHub] commons-compress issue #70: Use Java 7 Files.createTempDirectory instead of ...

2018-09-28 Thread coveralls 
Github user coveralls commented on the issue:

https://github.com/apache/commons-compress/pull/70
  

[![Coverage 
Status](https://coveralls.io/builds/19243727/badge)](https://coveralls.io/builds/19243727)

Coverage remained the same at 86.781% when pulling 
**bb6890f0fe9bc51a9e493d9ee9ab02a04a155ac5 on DaGeRe:master** into 
**ac3d73153725fa1bce721055c4ac61423570d34c on apache:master**.



---