dense-analysis / ale

Check syntax in Vim/Neovim asynchronously and fix files, with Language Server Protocol (LSP) support
BSD 2-Clause "Simplified" License
13.48k stars 1.43k forks source link

Ensure ale_filename_mappings option works optimally #4814

Open w0rp opened 1 month ago

w0rp commented 1 month ago

It's time we do another pass of the codebase to do the following:

  1. Simplify code for running linters and fixers so they use the formatted paths and executables everywhere.
  2. Simplify tests further.

It may be time to add a command such as AssertFixer to make writing tests for fixers easier, and it may also be time to bite the bullet and update the Vader version we run for tests, as we fixed the version a long time ago.

In addition, we need to run over the LSP code and tsserver code, and make double sure the ale_filename_mappings option is functioning with Docker Compose workflows properly.

w0rp commented 1 month ago

@neersighted This issue might pique your interest!