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

How do I include my deps in the plt? #16

Open
RobinClowers opened this issue May 16, 2016 · 3 comments
Open

How do I include my deps in the plt? #16

RobinClowers opened this issue May 16, 2016 · 3 comments

Comments

@RobinClowers
Copy link

I'm getting a bunch of errors like Callback info about the 'Elixir.Mix.Task' behaviour is not available. Is there a flag to include my deps when building the ptl?

@fishcakez
Copy link
Owner

The plugin relies on OTP application information to discover deps. This means a dep must be in the applications list in a mix.exs file to be added to the PLT. If the application requires another application then it should be in applications. I can see this might be problematic in the case of mix tasks where the application does not require mix but there is an associated mix task. I believe https://github.com/jeremyjh/dialyxir is more flexible as it relies on the user to config the PLT rather than automatic generate from OTP metadata

I would be happy to accept a PR that added the ability to add extra applications to the PLT via the mix.exs project/0 keyword metadata.

@RobinClowers
Copy link
Author

Hmm, that would explain the mix error, but I'm also seeing errors about plug, which is in our applications list. Any idea why that wouldn't get picked up?

@fishcakez
Copy link
Owner

The latest tag only uses applications of the project in the PLT so if you have phoenix listed in top level mix.exs but not plug (listed in phoenix's mix.exs) there will be warnings. If plug is listed in the top level mix.exs I have no idea why it wouldnt get picked up. I believe the previous tag to last will pick up applications recursively, which is better behaviour but can cause memory issues for some people.

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

2 participants