Skip to content

Add support for a configuration file #60

@axelson

Description

@axelson
Member

This is in addition to the workspace/didChangeConfiguration option.

Could look at erlang-ls/erlang_ls#61 for inspiration

Activity

znorris

znorris commented on Apr 6, 2023

@znorris
Contributor

I've run into a case where I believe adding support for a configuration file would prove helpful.

I recently resolved an issue where :yamerl was incorrectly throwing an error while I was debugging my application. The fix was adding an excludedModules key to the Elixir LS configuration. (Thanks for having the note about NIFs in the README)

"excludeModules": [
    ":yamerl"
]

Being able to exclude this any time the project is running Elixir LS would be very helpful as it will give a deliberate place to share this knowledge among other developers.

I'm using the VS Code extension which allows me to save the exclusion in our repo via the .vscode/launch.json file. However, the exclusion has to be duplicated among VS Code launch configurations, and obviously it won't be available to non VS Code devs.

Thanks for what you do! 🥳

lukaszsamson

lukaszsamson commented on Jul 19, 2024

@lukaszsamson
Collaborator

I don't see much value in having a separate config mechanism.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      Participants

      @axelson@znorris@lukaszsamson

      Issue actions

        Add support for a configuration file · Issue #60 · elixir-lsp/elixir-ls