[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

[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

[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

[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

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

[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