Github user jvanzyl commented on the pull request:

    https://github.com/apache/maven/pull/21#issuecomment-45966467
  
    Most excellent, thank you. Do you think you might also try your hand at 
creating an integration test and trying to assert the change doesn't break 
anything else in Maven? You can use the script at the bottom of [1] as a guide. 
I will try to get it processed this weekend but if you can work on an 
integration test I will be able to process it that much faster. Thanks for 
turning the patch into a pull request, it makes integrating any change much 
easier.
    
    [1]: http://takari.io/2014/06/02/contributing-to-maven-core.html


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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

Reply via email to