Skip to content

fix: use :peer for cluster tests#207

Merged
josevalim merged 6 commits intophoenixframework:mainfrom
davydog187:peer
Feb 5, 2026
Merged

fix: use :peer for cluster tests#207
josevalim merged 6 commits intophoenixframework:mainfrom
davydog187:peer

Conversation

@davydog187
Copy link
Contributor

This is logging a warning on Elixir 1.19.3

This is logging a warning on Elixir 1.19.3
@SteffenDE
Copy link
Contributor

I guess we can increase the minimum required elixir version

@josevalim
Copy link
Member

Yes, let's bump both Erlang and Elixir and we can ship it.

@davydog187
Copy link
Contributor Author

davydog187 commented Feb 5, 2026

@josevalim i copied over the CI workflow from phoenix live view and changed the minimum supported version to OTP 25, which is the earliest that supports :peer.

For the library itself, I see no reason to change since this is just a test dependency. Let me know if you want to change, its currently Elixir 1.6

@josevalim
Copy link
Member

If we don't test it on v1.6, then we shouldn't say we support it. We can likely update it to ~> 1.13.

davydog187 and others added 2 commits February 5, 2026 18:22
Co-authored-by: José Valim <jose.valim@gmail.com>
@davydog187
Copy link
Contributor Author

@josevalim done

@josevalim josevalim merged commit 0b63dce into phoenixframework:main Feb 5, 2026
5 checks passed
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

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