Skip to content

Bump Elixir to 1.20.2 and OTP 28.5.0.3#6437

Open
zoldar wants to merge 47 commits into
masterfrom
bump-elixir-1-20
Open

Bump Elixir to 1.20.2 and OTP 28.5.0.3#6437
zoldar wants to merge 47 commits into
masterfrom
bump-elixir-1-20

Conversation

@zoldar

@zoldar zoldar commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Changes

This PR updates Elixir to 1.20.2 and OTP to latest stable 28 (28.5.0.3).

Some minor and patch updates of dependencies were done in the process - either to meet the minimum requirements for the compatibility with new Elixir and OTP versions or in response to security advisories reported via mix deps.get.

The most prominent change is removal of Bamboo.SMTPAdapter which is no longer maintained and fails to compile under Elixir 1.20+.

A .dialyzer_ignore.exs file was introduced ignoring certain errors which either cannot be easily resolved (the ones related to the opaque type) or seem downright bogus (the one in modal appeared out of the blue late in the conversion process).

Some credo rules got relaxed as they were too stringent.

Changelog

  • Entry has been added to changelog

@zoldar zoldar changed the title [WIP] Bump elixir 1 20 [WIP] Bump to elixir 1.20 Jun 8, 2026
@zoldar zoldar force-pushed the bump-elixir-1-20 branch 2 times, most recently from f80405e to 3e6974b Compare June 10, 2026 07:04
@zoldar zoldar changed the title [WIP] Bump to elixir 1.20 [WIP] Bump to elixir 1.20.1 and OTP 28.5.0.1 Jun 10, 2026
@zoldar zoldar force-pushed the bump-elixir-1-20 branch 5 times, most recently from a62d39f to 6effbe2 Compare June 10, 2026 10:05
@zoldar zoldar added the preview label Jun 10, 2026
@github-actions

Copy link
Copy Markdown
Preview environment👷🏼‍♀️🏗️
PR-6437

@zoldar zoldar changed the title [WIP] Bump to elixir 1.20.1 and OTP 28.5.0.1 Bump Elixir to 1.20.1 and OTP 28.5.0.1 Jun 10, 2026
@zoldar zoldar marked this pull request as ready for review June 10, 2026 10:47
@zoldar zoldar requested a review from a team June 10, 2026 10:48

@cnkk cnkk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While we're at it modifying the Dockerfile, let's pin the images as well.
IMO it's okay to give up ALPINE_VERSION.

I extracted the meta digest with docker buildx imagetools inspect alpine:3.22.4, so the Dockerfile should work on multi arch build automatically.

Comment thread Dockerfile Outdated
Comment thread Dockerfile Outdated
Comment thread Dockerfile Outdated
@zoldar zoldar force-pushed the bump-elixir-1-20 branch from c5db0fc to c99c8ac Compare July 7, 2026 10:24
@zoldar zoldar changed the title Bump Elixir to 1.20.1 and OTP 28.5.0.1 Bump Elixir to 1.20.2 and OTP 28.5.0.3 Jul 7, 2026
@zoldar zoldar force-pushed the bump-elixir-1-20 branch 3 times, most recently from 8b38584 to dd13ee3 Compare July 7, 2026 13:12
zoldar and others added 26 commits July 7, 2026 15:14
Co-authored-by: Cenk Kücük <git@cenk.me>
@zoldar zoldar force-pushed the bump-elixir-1-20 branch from dd13ee3 to 8175c8c Compare July 7, 2026 13:14
@zoldar zoldar requested review from aerosol and cnkk July 7, 2026 13:25
setup do
# mock all domains resolve
Plausible.DnsLookup.Mock
|> expect(:lookup, fn _domain, _type, _record, _opts, _timeout ->

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aerosol For some reason this mock started to be called twice instead of once since bumping deps and I'm not sure why.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants