Allow public compare/2 functions when used in Enum.sort

exercism / elixir-analyzer

GNU Affero General Public License v3.0

30 stars 32 forks source link

Allow public compare/2 functions when used in Enum.sort #289

Closed angelikatyborska closed 2 years ago

angelikatyborska commented 2 years ago

Closes https://github.com/exercism/elixir-analyzer/issues/283

@jiegillet I hope this doesn't conflict with your work. You said you wanted to tackle this issue after https://github.com/exercism/elixir-analyzer/issues/276 and I figured I can help out because you didn't start yet, right?

angelikatyborska commented 2 years ago

I noticed that my code doesn't handle aliasing... I will merge it anyway now, and we can improve in a separate PR.

Do you know if it would be possible to somehow reuse the alias tracking that's defined in ElixirAnalyzer.ExerciseTest.AssertCall.Compiler? It would be a shame to have to write the same thing again.