Fixes #11726 - correct the overloaded method order

[dancrumb]

No description provided.

[codecov]

Codecov Report

Merging #11727 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master   #11727      +/-   ##
==========================================
+ Coverage   96.26%   96.26%   +<.01%     
==========================================
  Files          94       94              
  Lines        9190     9191       +1     
==========================================
+ Hits         8847     8848       +1     
  Misses        343      343

Impacted Files	Coverage Δ
lib/query-interface.js	92.19% <ø> (ø)	⬆️
lib/sequelize.js	95.93% <ø> (ø)	⬆️
lib/model.js	96.54% <100%> (ø)	⬆️
lib/dialects/mssql/query.js	95.53% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c1e87cd...1973e5a. Read the comment docs.

[SimonSchick]

I am not experiencing this issue at the moment, can you please provide a repro before we merge this?

[dancrumb]

Sure; there's one in the attached issue #11726

[dancrumb]

In addition, the discussion the led me to this conclusion is available in this issue: microsoft/TypeScript#35472

[papb]

I'm convinced, I think this should be merged, even if this was just a refactor (which it is not), since I agree that more specific overloads should come first (even for readability only). And surely merging this can't be harmful, can it?

@SimonSchick can we merge this?