zed icon indicating copy to clipboard operation
zed copied to clipboard

Enable additional Elixir Language Server configuration

Open mattbaker opened this issue 1 year ago • 6 comments

Check for existing issues

  • [X] Completed

Describe the feature

ElixirLS provides the ability to disable dialyzer, but I can't see a way to pass that configuration to Zed.

Is this possible? If so, we should update the ElixirLS docs. If not this would be a feature request.

If applicable, add mockups / screenshots to help present your vision of the feature

No response

mattbaker avatar Jan 19 '24 21:01 mattbaker

I was going to open a similar issue but found this one first.

Dialyzer is slow and not used by every project. Having the possibility of disabling it would be great.

crbelaus avatar Jan 27 '24 12:01 crbelaus

The setting in question: https://elixir-lsp.github.io/elixir-ls/features/#dialyzer-integration

mattbaker avatar Jan 31 '24 22:01 mattbaker

Don't have a direct answer, but I recently was able to turn on completions for the next-ls language server by adding this to my settings.json

"lsp": {
  "next-ls": {
    "initialization_options": {
      "experimental": {
        "completions": {
          "enable": true
        }
      }
    }
  }
}

Maybe for elixirLS you can experiment with something similar.

"lsp": {
  "elixir-ls": {
    "initialization_options": {
      ...
    }
  }
}

joetrimble avatar Feb 22 '24 01:02 joetrimble

@joetrimble Yeah I was hoping that would work too! I've tried various permutations and none of them seemed to have any affect unfortunately, but it was a good idea

mattbaker avatar Feb 22 '24 19:02 mattbaker

I have a branch that implements LSP workspace configuration which works but isn't complete, I got stuck on some rust-specific stuff that I couldn't understand and chat CPT was no help at all!

https://github.com/moomerman/zed/tree/workspace-configuration

this allows you to add config like this in Zed's settings.json

  "lsp": {
    "elixir-ls": {
      "workspace_configuration": {
        "elixirLS": {
          "dialyzerEnabled": false
        }
      }
    }
  }

moomerman avatar Feb 26 '24 19:02 moomerman

@moomerman I don’t know Rust but maybe you could open a PR and someone could help get it over the finish line? This is awesome!

mattbaker avatar Mar 05 '24 18:03 mattbaker