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

Unable to install com.cnuernber/ham-fisted with Neil 0.1.59 #177

Closed
teodorlu opened this issue Mar 22, 2023 · 2 comments
Closed

Unable to install com.cnuernber/ham-fisted with Neil 0.1.59 #177

teodorlu opened this issue Mar 22, 2023 · 2 comments

Comments

@teodorlu
Copy link
Contributor

I'm seeing this CLI behavior:

$ ls
$ neil dep search ham-fisted
:lib com.cnuernber/ham-fisted :version "1.000-beta-90" :description nil
$ neil dep add com.cnuernber/ham-fisted
$ ls
deps.edn
$ cat deps.edn
{:deps {}
 :aliases {}}

I'd expect neil dep add to add the dependency when we can find it with neil dep search.

@teodorlu
Copy link
Contributor Author

From @borkdude on Slack:

I suspect it has to do with neil not adding non-stable versions by default
but if you specify the :version yourself, then it will work

Teodor:

ahh, you're right. setting version explicitly adds everything as expected. neil dep search ham-fisted | fzf | xargs neil dep add also just works.
Any idea on what kind of semantics we want here?
"Do nothing, print nothing and return :exit 0" kind of surprised me.

@teodorlu
Copy link
Contributor Author

Fixed with #178 , closing.

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

No branches or pull requests

1 participant