Skip to content

fix: ensure Elixir 1.20 compatibility#251

Open
mikehostetler wants to merge 2 commits intomainfrom
codex/elixir-1-20-day1
Open

fix: ensure Elixir 1.20 compatibility#251
mikehostetler wants to merge 2 commits intomainfrom
codex/elixir-1-20-day1

Conversation

@mikehostetler
Copy link
Copy Markdown
Contributor

Summary

  • fix Jido-owned Elixir 1.20 warnings and the test cleanup needed for ExUnit 1.20
  • declare optional internal signal extensions explicitly so use Jido.Signal stays warning-free under 1.20
  • add GitHub Actions coverage for Elixir 1.20.0-rc.4 on OTP 28, including lint/quality validation

Why

Elixir 1.20 surfaces a few stricter warnings and test-behavior changes that do not show up on the current baseline. This keeps Jido source-compatible on day one and adds CI coverage so we keep verifying that path.

Validation

  • mise exec elixir@1.20.0-rc.4-otp-28 -- env MIX_BUILD_ROOT=_build-elixir-1.20-day1-test-clean mix test
  • mise exec elixir@1.20.0-rc.4-otp-28 -- env MIX_BUILD_ROOT=_build-elixir-1.20-day1-quality-clean mix q

Notes

  • Fresh Elixir 1.20 builds still print warnings from third-party dependencies, but Jido itself is warning-clean and the full quality suite passes.
  • This PR supersedes the earlier source-only compatibility branch by including CI coverage too.

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