merge exact matches of overlapping suggestion about cargo-spellcheck

[drahnr]

It is! It actually breaks a few baked in assumptions that changes are atomic and non-overlapping. So the second one can mess up the text after the first got applied with a different character span. Note that the languagetool got replace by nlprules backend, and the priority is more on the correctness of applying changes.
Stage 2 would be, to re-reparse comments that had overlapping fixes, but that's not really in the scope of 🔰 anymore.

I'd recommend to take a look at the structure how it works, do a first draft PR, might be just comments!, where you think the changes should go and we can continue the discussion from there :)

[drahnr]

In the hindsight, it's probably a non trivial issue to disambiguate (smells like coverage problem on spans).

The 🔰 part is to detect overlaps and discard the secondary, but do another cycle of checking with the provided checkers.

[granddaifuku]

@drahnr
If this is still the issue, can I work on the 🔰 part of this at a minimum?
Thanks