ARTEMIS-5210 bump com.google.errorprone:error_prone_core from 2.31.0 to 2.36.0

[jbertram]

No description provided.

[clebertsuconic]

What these other changes have to do with the component upgrade?

Can they be separate commits with a better description?

[jbertram]

@clebertsuconic, these change are required because running with -Perrorprone will fail without them since the duplicate branches are marked as errors.

[gemmellr]

I'm fine with the other changes, they make sense overall, but are only being done due to the upgrade so it seems reaosonable enough to bundle them. I just think the commented-out-code shouldnt be there, better nothing.

I did also type and delete 'maybe we could disable/downgrade-to-warning that new check?' before suggesting just to delete the comment since I'm fine with the changes. I'm also fine with effetively-same/duplicate branches where its clearer though, so I'd be fine without the check also.

[clebertsuconic]

@jbertram sure.. delete it.

[jbertram]

@clebertsuconic, done!