Skip to content

[codex] Fix Elixir 1.20 compile warnings - #33

Merged
mikehostetler merged 2 commits into
mainfrom
codex/elixir-1-20-otp29-20260612
Jun 12, 2026
Merged

[codex] Fix Elixir 1.20 compile warnings#33
mikehostetler merged 2 commits into
mainfrom
codex/elixir-1-20-otp29-20260612

Conversation

@mikehostetler

Copy link
Copy Markdown
Contributor

What changed

  • Renamed the dead-letter record() type to avoid overriding Elixir's built-in type.
  • Removed unused logger requirements.
  • Collapsed unreachable fallback clauses into internal case branches where defensive behavior is still useful.
  • Reordered nil module normalization before the generic atom clause.

Why

Elixir 1.20 / OTP 29 surfaces these as compile warnings, and CI treats warnings as errors.

Validation

  • MIX_ENV=test mix compile --warnings-as-errors
  • Final full package sweep across the local monofolder passed under Elixir 1.20 / OTP 29.

@mikehostetler
mikehostetler marked this pull request as ready for review June 12, 2026 12:31
@mikehostetler
mikehostetler merged commit 807821b into main Jun 12, 2026
18 checks passed
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.

1 participant