rnveach
rnveach
I'm sorry I forgot I was suppose to do something here. I don't have time today to look into it more and make a fix. If you can do something...
Not a maintainer of this repo, but... > if the method is called with null arguments welcome(18, null), it throws NPE, however my mutation score still shows up 100% even...
> As we use Sets/ TreeSets/ Lists, we must make sure that equals and hashcode are overridden Nobody has to override such methods if objects are immutable or you do...
@romani @Vyom-Yadav @nrmancuso My recommendation is to drop this PR and proceed with recommendation at https://github.com/checkstyle/checkstyle/issues/12243 . We should also need to worry less about hash code, and just implement...
Just pointing out that this wording was probably copied from https://checkstyle.org/config_filters.html#SuppressWithNearbyCommentFilter_Properties which has a similar property name.
> range I assume the full name will be `rangeFormat`. I am ok with it. I don't have anything else really.
According to documentation of similar filter, https://checkstyle.org/config_filters.html#SuppressWithNearbyCommentFilter_Examples , the property can be other things like `$2` to pull in the range from the suppression message. Ex: `CHECKSTYLE IGNORE (\w+) FOR...
We are turning off the loggers for this test, never turning them back on, and it is most likely leaking into other tests and not guaranteed that we are turning...
@Tiku24 I do not suggest this issue for someone new. Please read https://github.com/checkstyle/checkstyle/wiki/To-student-of-GSoC-2020 .
@developernarendra Since you are new, find another issue with an easy label.