Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Mix needs to validate the application callback #685

Closed
manpages opened this Issue Dec 2, 2012 · 4 comments

Comments

Projects
None yet
3 participants
Contributor

manpages commented Dec 2, 2012

Consider the following typo in mix.exs:

def application do
  [
    applications: [],
    mod: {MyApp}
  ]
end

it is rather hard to trace (at least when a programmer faces it for the first time).

It would be great if Mix could validate the application callback to minimize time spent to find such silly mistakes.

Contributor

yrashk commented Dec 2, 2012

👍 on this one

Owner

josevalim commented Dec 2, 2012

Agreed. If someone can send a pull request, it would be much appreciated.

Contributor

yrashk commented Dec 2, 2012

I might take a stab at it.

On Sun, Dec 2, 2012 at 3:40 AM, José Valim notifications@github.com wrote:

Agreed. If someone can send a pull request, it would be much appreciated.


Reply to this email directly or view it on GitHubhttps://github.com/elixir-lang/elixir/issues/685#issuecomment-10928624.

Contributor

yrashk commented Dec 2, 2012

Got this working, but no tests yet. How extensively should we test this?

yrashk added a commit to yrashk/elixir that referenced this issue Dec 2, 2012

@yrashk yrashk closed this in cf5f57f Dec 2, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment