Am 12.02.2018 um 18:52 schrieb Gary Gregory:
I agree 100% and will proceed. I thought about it overnight and it does not
make sense to leave a mix of abstract classes and interfaces in
StrSubstitutor.
+1, but
please revert "Update actual Checkstyle from 6.19 to 8.8.", as
Checkstyle 7+ requries Java 8+.
and please fix the findbugs violation:
[INFO] --- findbugs-maven-plugin:3.0.5:check(default-cli)@ commons-text---
[INFO] BugInstance size is 1
[INFO] Error size is 0
[INFO] Total bugs: 1
[INFO] org.apache.commons.text.StringTokenizer.clone() does not call
super.clone() [org.apache.commons.text.StringTokenizer] At
StringTokenizer.java:[lines 1138-1140] CN_IDIOM_NO_SUPER_CALL
to fix the travis build.
Thanks,
Pascal