-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bug: alias lifting produces broken code when aliases conflict with other existing usages #193
Comments
thanks for the report! definitely something styler's supposed to notice, sorry that it's got a bug there |
hey @jsw800 , i wasn't able to reproduce the issue with the snippet you gave me: https://github.com/adobe/elixir-styler/compare/me/fix-193?expand=1 tests still pass. is there more context you can provide? |
Ah sorry about that - I had kinda changed it from my original code so as to not share my actual source code. It looks like the difference between my original code and the example I shared is that defmodule SomeModule do
@moduledoc false
def do_something(token) do
# Guardian.Token.Jwt is a library module in the guardian library
# MyApp.Something.Guardian is a module defined inside my application
Guardian.Token.Jwt.decode_token(MyApp.Something.Guardian, token)
end
def do_something_else do
Application.get_env(:my_app, MyApp.Something.Guardian)
end
end |
ty, got it now :) |
Versions
Example Input
Before running
mix format
:Stacktrace / Current Behaviour
After running mix format:
The text was updated successfully, but these errors were encountered: