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

Add crypto to extra applications #100

Closed
philipgiuliani opened this issue Jul 7, 2023 · 3 comments · Fixed by #109
Closed

Add crypto to extra applications #100

philipgiuliani opened this issue Jul 7, 2023 · 3 comments · Fixed by #109

Comments

@philipgiuliani
Copy link
Contributor

Since this application uses crypto, it should probably be added to the list of extra_applications.

2023-07-07T19:32:59.543 helix_lsp::transport [ERROR] next-ls err <- "warning: :crypto.strong_rand_bytes/1 defined in application :crypto is used by the current application but the current application does not depend on :crypto. To fix this, you must do one of:\n"
2023-07-07T19:32:59.543 helix_lsp::transport [ERROR] next-ls err <- "\n"
2023-07-07T19:32:59.543 helix_lsp::transport [ERROR] next-ls err <- "  1. If :crypto is part of Erlang/Elixir, you must include it under :extra_applications inside \"def application\" in your mix.exs\n"
2023-07-07T19:32:59.543 helix_lsp::transport [ERROR] next-ls err <- "\n"
2023-07-07T19:32:59.543 helix_lsp::transport [ERROR] next-ls err <- "  2. If :crypto is a dependency, make sure it is listed under \"def deps\" in your mix.exs\n"
2023-07-07T19:32:59.543 helix_lsp::transport [ERROR] next-ls err <- "\n"
2023-07-07T19:32:59.544 helix_lsp::transport [ERROR] next-ls err <- "  3. In case you don't want to add a requirement to :crypto, you may optionally skip this warning by adding [xref: [exclude: [:crypto]]] to your \"def project\" in mix.exs\n"
2023-07-07T19:32:59.544 helix_lsp::transport [ERROR] next-ls err <- "\n"
2023-07-07T19:32:59.544 helix_lsp::transport [ERROR] next-ls err <- "  lib/next_ls.ex:508: NextLS.token/0\n"
2023-07-07T19:32:59.544 helix_lsp::transport [ERROR] next-ls err <- "\n"
@mhanberg
Copy link
Collaborator

mhanberg commented Jul 8, 2023

I'll look into this, thanks!

Side note, I would love it if you'd submit a pull request to the README with the config you used to add Next LS to helix 😄

@philipgiuliani
Copy link
Contributor Author

Hi! I am planning to do that, but its not working yet 🫣 I cant see any logs except of a language server timeout. i will investigate and document it!

@dvic
Copy link
Contributor

dvic commented Aug 5, 2023

@philipgiuliani any luck getting it to work with helix? :)

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