mirror of
https://github.com/dense-analysis/ale.git
synced 2025-12-06 12:44:23 +08:00
* add support for expert * Refactor elixir expert linter from feedback - Simplify configuration by removing executable lookup logic - Update documentation to reflect current configuration options - Rename test file and update assertions for the new configuration * default to expert for the lsp The install instructions included something I missed. It includes a global install, where the default executable is just `expert`. I'll default to that and review if I should should add a configuration for a global config. --------- Co-authored-by: Your Name <pmonson711@nfiindustries.com>
21 lines
608 B
Plaintext
21 lines
608 B
Plaintext
Before:
|
|
call ale#assert#SetUpLinterTest('elixir', 'expert')
|
|
|
|
After:
|
|
call ale#assert#TearDownLinterTest()
|
|
|
|
Execute(should set correct defaults):
|
|
AssertLinter 'expert', ale#Escape('expert')
|
|
|
|
Execute(The executable should be configurable):
|
|
let b:ale_elixir_expert_executable = 'foobar'
|
|
|
|
AssertLinter 'foobar', ale#Escape('foobar')
|
|
|
|
Execute(should set correct LSP values):
|
|
call ale#test#SetFilename('../test-files/elixir/umbrella_project/apps/app1/lib/app.ex')
|
|
|
|
AssertLSPLanguage 'elixir'
|
|
AssertLSPOptions {}
|
|
AssertLSPProject ale#path#Simplify(g:dir . '/../test-files/elixir/umbrella_project')
|