Treating warnings as errors in Elixir's mix compile

Warnings in Elixir are usually an important sign of a problem in the codebase. There is an easy way to make them gone.

Table of contents

    Imagine that you define an alias to a module and then try to compile it:

    defmodule App.User do
    end
    
    defmodule App.Users do
      alias App.User
    end
    
    ...
    iex> mix compile
    iex> warning: unused alias User

    or you're trying to call a function on a non-existing module:

    defmodule App.Users do
      def fetch_user(id), do: User.get(id)
    end
    
    ...
    
    iex> mix compile
    iex> warning: User.get/1 is undefined (module User is not available or is yet to be defined)

    Both of these cases generate warnings. The latter is also indication of en error:

    defmodule App.Users do
      def fetch_user(id), do: User.get(id)
    end
    
    ...
    
    iex> mix compile
    iex> App.Users.fetch_user(1)
    ** (UndefinedFunctionError) function User.get/1 is undefined (module User is not available)
        User.get(1)

    Some of the warnings will not cause an error, and some will. From my perspective, you should not tolerate any of these.

    Treating warnings as errors in mix compile

    Here is an easy way to treat every warning as an error during project compilation:

    iex> mix compile --warnings-as-errrors
    iex> warning: User.get/1 is undefined (module User is not available or is yet to be defined)
    
    Compilation failed due to warnings while using the --warnings-as-errors option

    Thanks to --warnings-as-errors, we expect compilation to fail whenever there is a warning in the app, and that's exactly why we see this message:

    Compilation failed due to warnings while using the --warnings-as-errors option

    Setting warning as errors flag in the mix.exs

    Using the flag is cool, but you might forget to include it in every command that requires a project to be compiled. If you don't want to forget, here is an option that you can set in mix.exs:

    def project do
      [
        ...
        elixirc_options: [
          warnings_as_errors: true
        ]
      ]
    end

    From now on, whenever you run a command that compiles the project, the warnings_as_errors flag will be used.

    That's it! Happy coding đŸ€“

    Szymon Soppa Web Developer
    Szymon Soppa Curiosum Founder & CEO

    Read more
    on #curiosum blog

    Continuous integration (CI) Elixir

    Mastering Elixir CI pipeline

    Developer time is precious for business. Don't waste it on things that can be easily automated. One of these things is a part of the code review process - quality checks. Continuous integration (CI) is where this automation is put in place.