-
Notifications
You must be signed in to change notification settings - Fork 17
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
NullPointerException with getting started steps #218
Comments
More info: I had to add the following packages to the dev container to get this far:
Also, I get the same exception in a Calva REPL as well (if it helps to diagnose) |
Also, I get the same exception if I start a repl with noj instead of clay:
|
Many thanks, @alza-bitz. I will look soon. |
Hi @alza-bitz, Could you create a
This will prevent Clay from trying to automatically open the browser view. It seems it fails to do that from within the dev container. Then, you will be able to open the URL yourself (and may need to take care of port forwarding, depending on your setup). |
Probably duplication of #185 |
Thanks! |
Hi @daslu appreciate the quick response! I checked the docs here https://scicloj.github.io/clay/#configuration and with One suggestion I had based on experiences with other server apps running in dev containers, it seems that it's possible to detect this case on the server side by some rule, avoiding exceptions thrown and then the VS Code auto port mapping can work as expected. I'm not sure how this would be done (some judicious exception handling when trying to open a browser perhaps), but in theory this would avoid the need for a config setting and the manual port mapping, saving two steps. Thanks 🙏 |
@alza-bitz I'll leave the issue open so we can return to that when we have some time. |
Hi there,
I'm getting a
NullPointerException
when trying to follow the "getting started" steps here: https://scicloj.github.io/clay/#getting-startedThis is in a VSCode dev container with Java 17, running from a directory with no
deps.edn
or Clojure project, so the classpath is just clay and transitive deps (I presume):The text was updated successfully, but these errors were encountered: