Roman Ivanov
Roman Ivanov
Do you already have parameter by which we config pitest to skip mutation over `unmodifiableList` at all ? avoidCallTo do no mutation inside method, but still mutate call of method.
yes, you are right, we already use avoidCallsTo for classes. https://github.com/checkstyle/checkstyle/blob/c1ca1b903beaed94923f621fac2197394ae011dd/pom.xml#L3079-L3082 but did not want to exclude all methods from this class, as it has bunch of other methods like...
>This type of change is not good as it might lead to a problem. We did update to do extra loop only because this is nano optimization will not do...
In recent world, people do care about data protection more than memory, so this pattern is frequent and better to not raise flag on it. Extra config is completely ok,...
@hcoles , is there any plan to fix this issue ? or may be you just share a bit more links to code on how to do this (based on...
CI is source of truth, if it fails, we cannot merge. @Rohanraj123 , if this update is complicated, please abandon it and move to another issue.
@martin-mfg , I created new issue https://github.com/checkstyle/checkstyle/issues/9280 please update reference in your commit message to new issue.
@martin-mfg, please pause work on this for a bit, we are close to merge state for #7418 by @rnveach
@martin-mfg, please rebase, related PR is merged
please rebase to our latest code. I used your commit to make https://github.com/checkstyle/xwiki-commons/tree/checkstyle_9280 please update travis script to do `git checkout checkstyle_9280` to use your update in CI scripts. wercker...