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 to Elixir v1.16.0 and OTP 26 #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mbaleczny
Copy link

@mbaleczny mbaleczny commented Jan 31, 2024

Also formatted the code, resolved String.slice/2 warning appearing with Elixir v1.16-otp-26 and updated github workflows ci config file

@@ -5,7 +5,7 @@ defmodule ExMarcel.MixProject do
[
app: :ex_marcel,
version: "0.1.0",
elixir: "~> 1.12",
elixir: "~> 1.14",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what will happen with apps running 1.12?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apps running Elixir below 1.14 probably won't work. That's should be fine, because only the latest three Elixir releases along with the latest 3 OTP releases should be supported.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uhm could that break many applications? what would be the implicancies to relax the version? I guess it should work anyway...

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested it in an app with Elixir v1.12 OTP 24. It seems to be working. But there's a compilation warning
warning: the dependency :ex_marcel requires Elixir "~> 1.14" but you are running on v1.12.3.

Disclaimer - this is just my guess. I'm not 100% sure about this.
It may not be the case of this library, but Elixir Kernel and/or Erlang functions may differ. Relaxing the version may get someone using this library with older Elixir/OTP version and causing errors and/or crashes.

@michelson
Copy link
Member

thanks for the contribution, just added one question

@warmwaffles
Copy link

warmwaffles commented Apr 8, 2024

@michelson for elixir libraries anyone running older than 3 versions should be left behind. For exqlite I only support latest 3 versions of OTP and Elixir.

https://github.com/elixir-sqlite/exqlite/blob/8cac98612426d41672730d18511204eb23ce77a3/.github/workflows/ci.yml#L52-L53

I would say supporting back to 1.14 should be minimum and anyone on 1.12 needs to update as that does not compile with latest OTP at all and has severe security vulnerabilities.

@warmwaffles
Copy link

@michelson any news if we can get this bump to happen? The string slice warning is now appearing all over my tests.

@warmwaffles
Copy link

@michelson any updates?

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.

3 participants