Fix Ord of cmp_fix

[MichaReiser]

Summary

Fixes #12469

Test Plan

Running ruff with the given test snipped no longer panics when using rust nightly

Okay, I think the problem is the first return that higher prioritizes UnusedImport. The problem is that it violates

< is transitive: a < b and b < c implies a < c. The same must hold for both == and >.

Let's say we have

• a: RedefinedWhileUnused with a min fix of 100
• b: UnusedImport with a min fix of 0
• c: any other rule with a min fix of 5

then:

• a < b because of (Rule::RedefinedWhileUnused, Rule::UnusedImport) => std::cmp::Ordering::Less,
• b < c because of .then_with(|| fix1.min_start().cmp(&fix2.min_start()))
• but a > c .then_with(|| fix1.min_start().cmp(&fix2.min_start()))
  Ord violation panic when fixing file with nightly compiler #12469 (comment)

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.