Skip to content

Bump Elixir to test against to 14.2 #32

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

Merged
merged 1 commit into from
Jan 13, 2023
Merged

Bump Elixir to test against to 14.2 #32

merged 1 commit into from
Jan 13, 2023

Conversation

sato11
Copy link
Contributor

@sato11 sato11 commented Jan 13, 2023

Hey can we perhaps now bump to 1.10 and 1.14?
https://hexdocs.pm/elixir/main/compatibility-and-deprecations.html

Last change was #31 where minimum required version remained 1.7. On the other hand since plug specifies 1.10 it'll feel better if we can follow it, won't it? This dropped in favor of #32 (comment).

Other changes included is what I have validated in fork. Basically it's how plug configures it.

@josevalim
Copy link
Member

We don't need to follow Plug, as this updates less frequently, so I would stay with ~> 1.7, especially as it is a critical dependency. :)

@sato11
Copy link
Contributor Author

sato11 commented Jan 13, 2023

Thanks! Great to know that and it makes sense to me 👋 Is it then alright to go only with 1.13 to 1.14?

@sato11 sato11 changed the title Bump required Elixir to currently supported version Bump Elixir to test against to 14.2 Jan 13, 2023
@sato11
Copy link
Contributor Author

sato11 commented Jan 13, 2023

Polished after the comment. Feel free to let me know if I seem to miss the point!

@josevalim josevalim merged commit 70a416d into elixir-plug:main Jan 13, 2023
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

@sato11
Copy link
Contributor Author

sato11 commented Jan 13, 2023

Hooray!

@sato11 sato11 deleted the bump-versions-to-test-against branch January 13, 2023 14:51
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.

2 participants