Github pull bot overwrites contents of PRs

Hey, I’m sorry if this is the wrong category, but it might be worth looking:

I tried to write my own quickfix for a rule you have here: Add quick fix for `Map.get(key)` `isEqualTo`/`isNotEqualTo(value)` by asm0dey · Pull Request #5607 · SonarSource/sonar-java · GitHub. The first couple of reviews were positive, but then something happened, and on the next review, a bot overwrote all of my changes, which is, supposedly, a bug in the pipeline

Hi @asm0dey,
Thanks for submitting this change and coming here to post about it. We usually recommend people come to this forum to explain the change they want to include in the analyzer.

Regarding your changes and closed PR, we are not sure what happened but you might want to check your integration with pull bot that might have misfired.

We will be having a look at your new PR in the coming days.

Dorian

Well, there is already an inspection. I just implemented a quick fix for it :slight_smile:

Anyways, the reviewer proposed new changes, and I’ve submitted the new PR with them implemented: fix(S5838): wire quick fix for Map.get isEqualTo/isNotEqualTo by asm0dey · Pull Request #5608 · SonarSource/sonar-java · GitHub

I have several more quick fixes like this one for AssertJ, working for your other inspections if this one is good enough.

And the same happened again here fix(S5838): wire quick fix for Map.get isEqualTo/isNotEqualTo by asm0dey · Pull Request #5608 · SonarSource/sonar-java · GitHub