Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
From: kinow <git@git.apache.org>
To: issues@commons.apache.org
Reply-To: issues@commons.apache.org
References: <git-pr-133-commons-lang@git.apache.org>
In-Reply-To: <git-pr-133-commons-lang@git.apache.org>
Subject: [GitHub] commons-lang pull request: Add tests for missed branches in
 DateUt...
Content-Type: text/plain
Message-Id: <20160424005624.F06C8DFE2E@git1-us-west.apache.org>
Date: Sun, 24 Apr 2016 00:56:24 +0000 (UTC)

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