-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Set dialyxir and elixir_make to only: :dev #72
Conversation
Should we open an issue on Elixir about this? Weird we have never ran into this issue before. Also don't we still need |
f2c5480
to
17d222e
Compare
I updated this to use [:dev, :test] to fix travis |
But apps using this will use |
nerves_runtime doesn't need elixir_make or dialyxir in production |
2554247
to
78d49c2
Compare
If |
This is what i was wondering. Does building a dep not build in the env it is configured to run in? |
Not to beat a dead horse here, but the line 'compilers: [:elixir_make | Mix.compilers()],' also needs updating if |
…from getting into prod runtime_false does not propigate through hex. All hex packages are built using the production environment, therefore, changing the level of dependency to only dev will remove them from the application spec.
78d49c2
to
6f68955
Compare
This is ready now |
runtime_false does not propigate through hex. All hex packages are built using the production environment, therefore, changing the level of dependency to only dev will remove them from the application spec.