[GitHub] commons-lang issue #256: fix for LANG-1310

2017-03-19 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue:

https://github.com/apache/commons-lang/pull/256
  
Thanks! :+1: 


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


[GitHub] commons-lang issue #256: fix for LANG-1310

2017-03-18 Thread drajakumar
Github user drajakumar commented on the issue:

https://github.com/apache/commons-lang/pull/256
  
the comment on indentation from@jvz is fixed now.


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


[GitHub] commons-lang issue #256: fix for LANG-1310

2017-03-18 Thread coveralls
Github user coveralls commented on the issue:

https://github.com/apache/commons-lang/pull/256
  

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

Coverage increased (+0.04%) to 94.571% when pulling 
**2ef748c444b50ccce4819d876225cd134a903139 on drajakumar:master** into 
**39a2a6fa38c5e4d1293e00f98a89698a962d6775 on apache:master**.



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


[GitHub] commons-lang issue #256: fix for LANG-1310

2017-03-14 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue:

https://github.com/apache/commons-lang/pull/256
  
@drajakumar It's really great to get a pull request for this. :+1:  

I took a look at this issue some time ago, but gave up trying to solve it. 
Sadly I'm too tired for a review at the moment. I will get back to you in a few 
days, if nobody has reviewed/merged the pull request already.


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


[GitHub] commons-lang issue #256: fix for LANG-1310

2017-03-14 Thread coveralls
Github user coveralls commented on the issue:

https://github.com/apache/commons-lang/pull/256
  

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

Coverage increased (+0.06%) to 94.588% when pulling 
**6daa417fe9826d0bfa9c2366fc520a1519e8579f on drajakumar:master** into 
**39a2a6fa38c5e4d1293e00f98a89698a962d6775 on apache:master**.



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