[GitHub] commons-lang issue #191: ToStringExcludeNullValue

2017-04-28 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue:

https://github.com/apache/commons-lang/pull/191
  
Support for excluding `null` values was added in 
https://github.com/apache/commons-lang/commit/8147cc5b3de5fa7a3a3e8116355efa44367dc3c5

@gabriel-amaral Thanks for the pull request and sorry that it was not 
merged. Can you please close it? Thanks!




---
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 #191: ToStringExcludeNullValue

2016-09-15 Thread coveralls
Github user coveralls commented on the issue:

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

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

Coverage increased (+0.01%) to 93.587% when pulling 
**9f6acd29875b159f7f893671f0cf8c3934deb03d on amaral7:master** into 
**9c506279188a90dbbe124386cc52016682d17b5c 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 #191: ToStringExcludeNullValue

2016-09-25 Thread britter
Github user britter commented on the issue:

https://github.com/apache/commons-lang/pull/191
  
@amaral7 do you really want to configure this on a per field basis or would 
it be enough to add a configuration option to `ToStringBuilder` to configure 
the general behavior for all fields?


---
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 #191: ToStringExcludeNullValue

2016-09-26 Thread coveralls
Github user coveralls commented on the issue:

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

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

Coverage decreased (-0.01%) to 93.533% when pulling 
**9af3014b5188aa8c76b41e8dda0213eb7a542cb2 on amaral7:master** into 
**a04151819758fceb7f0cb855d279c9d46b2e9851 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 #191: ToStringExcludeNullValue

2016-09-26 Thread coveralls
Github user coveralls commented on the issue:

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

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

Coverage increased (+0.002%) to 93.547% when pulling 
**935149511e8e4f4fce5d4140ff68fd3196296436 on amaral7:master** into 
**a04151819758fceb7f0cb855d279c9d46b2e9851 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 #191: ToStringExcludeNullValue

2016-09-26 Thread amaral7
Github user amaral7 commented on the issue:

https://github.com/apache/commons-lang/pull/191
  
@britter when I made the changes, I was thinking to cover the largest 
number of possible cases, and for this reason I implemented this configuration 
per field that would cover even who wants the general behavior for all fields, 
but more painful. I think there may be cases that need this behavior per field, 
but I know that most cases would be more comprehensive, for this reason have 
made the changes. And that wasn´t so painful to me ;). 


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