-
Notifications
You must be signed in to change notification settings - Fork 50
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
Ease Elixir escripts execution by adding ${HOME}/.mix/escripts to path #153
Conversation
Hey, @btkostner. Thanks for this. I'm wondering if this is sufficiently generic though, to have included in the action. What do you think, @starbelly, @ericmj, @wojtekmach? |
+1 from me.
|
@btkostner, could you add a simple test that demonstrates the behaviour, and can be constantly used for regression testing? |
Co-authored-by: Paulo F. Oliveira <paulo.ferraz.oliveira@gmail.com>
Alright, I added it to the existing install test matrix, but if you want me to split it off to it's own job, I can do that. Tests show it working in Linux and Windows. Let me know if there is more that I should do. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good. Thanks @btkostner.
Merging to |
This adds the mix escripts folder to
PATH
, allowing you to runmix escript.install hex protobuf --force
and have access to the protobuf generator right away. Making life simpler.I'm having this as a WIP because I don't program in windows and need to confirm this actually works as intended.