Skip to content

chore(otp): build on OTP 29#60

Merged
Taure merged 4 commits into
mainfrom
chore/otp-29
Jun 30, 2026
Merged

chore(otp): build on OTP 29#60
Taure merged 4 commits into
mainfrom
chore/otp-29

Conversation

@Taure

@Taure Taure commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Draft — blocked on widgrensit/asobi#138.

OTP 29 deprecates catch Expr. asobi_lua's own code is catch-free; the work is toolchain + test deps:

  • .tool-versions erlang 29.0.2 (un-ignored + tracked, mirroring asobi/engine/saas), CI via version-file
  • meck ~> 0.9~> 1.2; test profile nowarn_deprecated_catch

Verified green on OTP 29.0.2 (compile, eunit 223/0, dialyzer) against asobi's chore/otp-29 branch.

Flip step once #138 merges: rebar3 upgrade asobi (pulls asobi main with kura v2.0.8), re-verify, mark ready. (CT is pre-existing broken on a .src sys_config consult — OTP-independent, out of scope.)

Taure added 3 commits June 30, 2026 11:02
OTP 29 deprecates `catch Expr` (hard error under warnings_as_errors).

- .tool-versions: erlang 29.0.2 (now tracked + un-ignored; drives CI via version-file)
- ci.yml / nightly.yml: use version-file '.tool-versions' instead of hardcoded otp-version
- meck ~> 0.9 -> ~> 1.2 (0.9 fails to compile on OTP 29)
- test profile: nowarn_deprecated_catch (test teardown uses catch; src is catch-free)

Verified green on OTP 29.0.2 against asobi's fixed branch: compile, eunit 223/0, dialyzer.

NOTE: the asobi pin must be bumped to main once widgrensit/asobi#138 merges
(asobi v2.0.8 kura). Draft until then.
Flip the asobi pin to main now that widgrensit/asobi#138 is merged.
Green on OTP 29.0.2: compile, eunit 223/0, dialyzer.
@Taure
Taure marked this pull request as ready for review June 30, 2026 10:05
cowboy 2.15.0 pins cowlib 2.16.1 transitively, which the GitHub Advisory
DB now flags (LOW, CVE-2026-43969). Promote cowlib to the patched ~> 2.17
(hex cowboy 2.15.0 is compatible with cowlib 2.17.1, per asobi_engine).
Real fix instead of an audit-ignore. eunit 223/0 on OTP 29.
@Taure
Taure merged commit c1f7676 into main Jun 30, 2026
15 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