elixir_sense icon indicating copy to clipboard operation
elixir_sense copied to clipboard

find elixir modules that require adding an alias

Open ajayvigneshk opened this issue 3 years ago • 3 comments

Basic code to find modules that require adding an alias. Sort of related to elixir-ls issue ~Marking it draft due to some known test failures that I would like to get some suggestions on how to handle them.~

ajayvigneshk avatar May 09 '22 03:05 ajayvigneshk

~Two of the tests fail as the newly added code suggests Kernel.SpecialForms as a new alias. I'm thinking this can be ignored from suggestions to fix. One other test failure can be fixed by ignoring the suggestions from the new code(I've used this approach in some tests already, not sure if it's okay though)~ I've updated code to be behind an opt

ajayvigneshk avatar May 09 '22 14:05 ajayvigneshk

@antedeguemon @lukaszsamson I've applied the suggestions and removed the usage of then due to failures for v 1.11.

ajayvigneshk avatar Oct 05 '22 09:10 ajayvigneshk

@antedeguemon @lukaszsamson Made a commit to fix credo errors

ajayvigneshk avatar Oct 05 '22 12:10 ajayvigneshk