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

chore: update Elixir, Erlang and dependencies #224

Merged
merged 19 commits into from
Jul 18, 2022
Merged

Conversation

polvalente
Copy link
Contributor

@polvalente polvalente commented Jul 17, 2022

This PR updates dependencies and deprecates support for OTP < 22, as well as Elixir < 1.10

@polvalente polvalente changed the title chore: update dependencies chore: update Elixir, Erlang and dependencies Jul 17, 2022
polvalente and others added 2 commits July 16, 2022 21:53
Co-Authored-By: David Bernheisel <david@bernheisel.com>
@polvalente polvalente mentioned this pull request Jul 17, 2022
@hkrutzer
Copy link
Contributor

OTP 25 would also be appreciated 🙂

@polvalente
Copy link
Contributor Author

OTP 25 would also be appreciated slightly_smiling_face

Nice catch! I've added it to the matrix, although it only works with Elixir 1.13+

@@ -9,7 +9,7 @@ An Elixir implementation of [gRPC](http://www.grpc.io/).

**WARNING: Be careful to use it in production! Test and benchmark in advance.**

**NOTICE: Erlang/OTP needs >= 20.3.2**
**NOTICE: Erlang/OTP needs >= 22 **
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this a hard requirement now? I didn't watch Erlang/Elixir version for a long time, so I may miss something. Ignore this if I miss the simple stuff :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The change in how erlang errors are matched drops OTP 21 support, which I don't see as a problem given how old it is

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