commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bodewig <...@git.apache.org>
Subject [GitHub] commons-compress issue #33: add-some-Unit-Tests Added some Unit Tests to inc...
Date Thu, 15 Jun 2017 15:31:11 GMT
Github user bodewig commented on the issue:

    https://github.com/apache/commons-compress/pull/33
  
    Thanks @TheRealHaui 
    
    I agree with your comment on `ChecksumCalculatingInputStreamTest#testGetValueThrowsNullPointerException`
this looks like a bug. So I'd rather remove the test and see the bug fixed. Do you want to
take a stab?
    
    Also, the Commons Compress community has decided to not use author tags, I'd have to strip
yours before merging the PR. Could you please remove them yourself if you modify this PR?
Feel free to add yourself as a contributor to the POM.


---
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.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message