Github user chonton commented on the issue:
https://github.com/apache/commons-lang/pull/331
closed with commit f56931c176fef5e164b681c740746aebdec3
---
Github user PascalSchumacher commented on the issue:
https://github.com/apache/commons-lang/pull/331
@chonton No further review comments by anyone, so I guess this is good to
merge?
---
Github user PascalSchumacher commented on the issue:
https://github.com/apache/commons-lang/pull/331
+1
---
Github user coveralls commented on the issue:
https://github.com/apache/commons-lang/pull/331
[![Coverage
Status](https://coveralls.io/builds/17069311/badge)](https://coveralls.io/builds/17069311)
Coverage decreased (-0.004%) to 95.229% when pulling
**b09d26f12bd4bfd1456
Github user coveralls commented on the issue:
https://github.com/apache/commons-lang/pull/331
[![Coverage
Status](https://coveralls.io/builds/17069311/badge)](https://coveralls.io/builds/17069311)
Coverage decreased (-0.004%) to 95.229% when pulling
**b09d26f12bd4bfd1456
Github user sebbASF commented on the issue:
https://github.com/apache/commons-lang/pull/331
The change is hard to review as it contains at least one unrelated change
and some code re-arrangement.
Would it be possible to provide a PR that addresses only the proposed
change?