jeremyjh / dialyxir

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

Bump minimal required Elixir version to 1.12.0 #516

Closed tombruijn closed 10 months ago

tombruijn commented 10 months ago

Since PR #514, the Kernel.then/2 macro is used, which was introduced in Elixir 1.12.0. If installed on older Elixir versions, it will fail to compile with the error below.

==> dialyxir
Compiling 66 files (.ex)

== Compilation error in file lib/dialyxir/project.ex ==
** (CompileError) lib/dialyxir/project.ex:365: undefined function then/2
    (elixir) expanding macro: Kernel.|>/2
    lib/dialyxir/project.ex:365: Dialyxir.Project (module)
    (elixir) expanding macro: Kernel.if/2
    lib/dialyxir/project.ex:365: Dialyxir.Project (module)

Bump the minimal required Elixir version to communicate older versions are no longer supported. Elixir 1.12 is also the oldest version tested in the CI.

If it wasn't the intention to drop support for older Elixir versions, please close this PR :)