search

jeremyjh / dialyxir

Mix tasks to simplify use of Dialyzer in Elixir projects.
Apache License 2.0
1.69k stars 141 forks source link

Add `:no-umbrella` config option #481

Closed pnezis closed 1 year ago

pnezis commented 1 year ago

In some cases you may want to place the lockfile in a parent folder without having an umbrella. With this extra option you avoid compilation at the parent folder which will fail since in such cases there is no mix.exs. If this flag is not set then the behaviour remains unchanged.

jeremyjh commented 1 year ago

Hi, thanks for contributing this, but having it as a CLI option seems a little bit inconvenient since it is a fact about the project configuration that will never change between invocations. I think it might be better to have it as a configuration option specified in the dialyzer options section of the project config in mix.exs.

pnezis commented 1 year ago

@jeremyjh you are right, updated the PR accordingly