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
@hauleth
Unfortunately, even with this, implementing mix deps.add in a foolproof way is impossible.
what if :deps is list within config/0?
what if it is module attribute instead of function?
what if there is no :deps key in config/0
what if :deps call function defined in another file?
Due to fact that mix.exs is executable it is hard.
That would be good enough for an external library then. Built in tool should work in 100% of the cases.
@AmNicd
I don't agree. You can make it bail out and give a good error message explaining why. If you've moved your deps away from the default, you're already probably experienced enough to deal with it. This is letting niche use cases prevent usability improvements from the majority.
The text was updated successfully, but these errors were encountered:
From twitter:
@hauleth
Unfortunately, even with this, implementing
mix deps.add
in a foolproof way is impossible.:deps
is list withinconfig/0
?:deps
key inconfig/0
:deps
call function defined in another file?Due to fact that
mix.exs
is executable it is hard.That would be good enough for an external library then. Built in tool should work in 100% of the cases.
@AmNicd
I don't agree. You can make it bail out and give a good error message explaining why. If you've moved your deps away from the default, you're already probably experienced enough to deal with it. This is letting niche use cases prevent usability improvements from the majority.
The text was updated successfully, but these errors were encountered: