Skip to content
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

Ensure ale_filename_mappings option works optimally #4814

Open
w0rp opened this issue Aug 2, 2024 · 1 comment
Open

Ensure ale_filename_mappings option works optimally #4814

w0rp opened this issue Aug 2, 2024 · 1 comment
Assignees

Comments

@w0rp
Copy link
Member

w0rp commented Aug 2, 2024

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 w0rp self-assigned this Aug 2, 2024
@w0rp
Copy link
Member Author

w0rp commented Aug 2, 2024

@neersighted This issue might pique your interest!

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

No branches or pull requests

1 participant