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

Bump minimal required Elixir version to 1.12.0 #516

Merged
merged 1 commit into from Aug 28, 2023

Conversation

tombruijn
Copy link
Contributor

@tombruijn tombruijn commented Aug 28, 2023

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 :)

Since PR jeremyjh#514, the [`Kernel.then/2`
macro](https://hexdocs.pm/elixir/Kernel.html#then/2) 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.
@tombruijn tombruijn changed the title Bump minimal required Elixir version Bump minimal required Elixir version to 1.12.0 Aug 28, 2023
@jeremyjh jeremyjh merged commit 7285f15 into jeremyjh:master Aug 28, 2023
12 checks passed
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

Successfully merging this pull request may close these issues.

None yet

2 participants