You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The docstring says Add a dependency to the project.clj file. so I thought it will only support lein, but I tried it on my deps.edn project and was pleasantly surprised it worked great 😄
Maybe change the docstring to Add a dependency to project.clj or deps.edn?
Also could maybe improve the error message a little in the case where it doesn't find the project file, maybe in cljr--add-project-dependency change it to something like:
The docstring says
Add a dependency to the project.clj file.
so I thought it will only support lein, but I tried it on my deps.edn project and was pleasantly surprised it worked great 😄Maybe change the docstring to
Add a dependency to project.clj or deps.edn
?Also could maybe improve the error message a little in the case where it doesn't find the project file, maybe in
cljr--add-project-dependency
change it to something like:so it gives a clear error message with the reason rather than just nil isn't stringp in
cljr--project-with-deps-p
The text was updated successfully, but these errors were encountered: