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
Fix formatting on Windows #375
Conversation
Bring formatting back on Windows. Regression in 0.6.0 Create elixir.yml try without downcase fix cwd Revert test without downcase fix matching fix capture fix missing end r
🤕 String.replace that takes a replacement function was added in Elixir 1.9 |
We've just dropped support for elixir 1.7 in 0.6. I wouldn't be happy with dropping 1.8 so soon. |
When Elixir 1.11 is out, you can probably drop 1.8. But now, yeah, I agree with you - hence the 🤕 . |
Gonna try a new idea. |
@@ -815,7 +815,7 @@ defmodule ElixirLS.LanguageServer.Server do | |||
|
|||
defp set_project_dir(%{project_dir: prev_project_dir, root_uri: root_uri} = state, project_dir) | |||
when is_binary(root_uri) do | |||
root_dir = SourceFile.path_from_uri(root_uri) | |||
root_dir = root_uri |> SourceFile.path_from_uri() |> Path.absname() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this normalizes the project_dir
path, so it's in "elixir case" (lower c:/
)
@lukaszsamson @axelson this patch should be mergeable now!
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! This looks good now. ❤️
Alternative PR to #373