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

feat(elixir): add Next LS #4626

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mhanberg
Copy link
Contributor

@mhanberg mhanberg commented Oct 5, 2023

Next LS is a new language server for Elixir

Homepage: https://www.elixir-tools.dev/next-ls
GitHub: https://github.com/elixir-tools/next-ls


Will write tests tomorrow, just getting this up before calling it a night.

@mhanberg
Copy link
Contributor Author

mhanberg commented Oct 5, 2023

@w0rp tests added, ready for review

@mhanberg
Copy link
Contributor Author

mhanberg commented Oct 5, 2023

failing test seems unrelated

Copy link
Member

@w0rp w0rp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you update documentation to list this as an available linter? See the :h ale-dev documentation.

Copy link

stale bot commented Mar 17, 2024

This pull request has been automatically marked as stale because it has not been updated recently. Make sure to write tests and document your changes. See :help ale-dev for information on writing tests.
If your pull request is good to merge, bother w0rp or another maintainer again, and get them to merge it.

@stale stale bot added the stale PRs/Issues no longer valid label Mar 17, 2024
@stale stale bot closed this Apr 22, 2024
@w0rp w0rp reopened this Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale PRs/Issues no longer valid
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants