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