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

Update asdf elixir version for gradual typing #353

Merged

Conversation

elliottneilclark
Copy link
Contributor

chore: update asdf elixir/otp versions

Summary:

  • The latest elixir has nice gradual typing. This PR updates to the
    latest version to bring that in.
  • Bump OTP as well

Test Plan:

  • make integration.kind
  • mix compile --force --warnings-as-errors

Summary:
- The latest elixir has nice gradual typing. This PR updates to the
  latest version to bring that in.
- Bump OTP as well

Test Plan:
- make integration.kind
- mix compile --force --warnings-as-errors
@elliottneilclark elliottneilclark force-pushed the elliott/bump_elixir_ver branch from 40388b0 to b3c0f3b Compare January 7, 2025 21:33
Summary:
- Add the latest elixir and otp version to the testing matrix. We've
  been running them at BatteriesIncluded for a while on the latest
  versions.
- Remove the restore keys field for plts. This causes us to re-generate
  new typespecs on new elixir versions (new otp versions as well but
  they don't seem to change the plts as much)

Test Plan:
- Green CI
@elliottneilclark elliottneilclark force-pushed the elliott/bump_elixir_ver branch from b3c0f3b to 8cd4293 Compare January 7, 2025 21:36
@elliottneilclark
Copy link
Contributor Author

The failure is a real dialyzer error that comes from the upgraded elixir.

Summary:
- Fix dialyzer error

Test Plan:
- CI
@mruoss mruoss merged commit 46a509d into coryodaniel:develop Jan 8, 2025
10 of 11 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.

2 participants