commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kinow <...@git.apache.org>
Subject [GitHub] commons-lang pull request: Add tests for missed branches in DateUt...
Date Sun, 24 Apr 2016 00:56:24 GMT
Github user kinow commented on the pull request:

    https://github.com/apache/commons-lang/pull/133#issuecomment-213860528
  
    Tested locally. All tests pass. Line coverage for Date keeps in 95%, branch coverage went
from 90% to 92%, having 194/210 whereas it previously was 189/210.
    
    Couldn't merge with the normal commands locally, as the remote branch was called 'master'
and git wasn't happy about that. Didn't have enough git-fu for a workaround, so merged manully
as a patch (see the bottom of the page at http://wiki.apache.org/commons/UsingGIT).
    
    Thanks for your contribution.
    Bruno


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