Skip to content

fix: bump the reusable workflow's elp-version default to the OTP 29 release - #73

Merged
Taure merged 1 commit into
mainfrom
fix/elp-version-default
Aug 11, 2026
Merged

Taure merged 1 commit into
mainfrom
fix/elp-version-default

Conversation

@Taure

@Taure Taure commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Third and final piece of the OTP 29 ELP fix.

#71 bumped the version default in elp/action.yml, but the reusable workflow carries its own elp-version input with its own default, and that one wins - every ELP job passes version: ${{ inputs.elp-version }} unconditionally, so the action's default is unreachable when called through the workflow.

Net effect on widgrensit/asobi#436, with the pin already at v2.1.5:

##[error]ELP 2026-02-27 ships no linux x86_64 binary for OTP 29.

The error is accurate and actionable now (an improvement on the old generic one), but it names the release #71 meant to stop using.

This bumps the workflow default to 2026-08-10 to match, with a comment noting the two must track each other since nothing enforces it. Same caveat as #71: that release ships no OTP 26 binary, so a consumer still on 26 must pass elp-version: '2026-02-27'.

…elease

#71 moved the default in elp/action.yml but not the one in the reusable
workflow, and the workflow's is the one that counts: every ELP job passes
`version: ${{ inputs.elp-version }}` unconditionally, so the action's default
is unreachable from there. Consumers therefore still resolved 2026-02-27 and
still failed with "ships no linux x86_64 binary for OTP 29" - the error was
now accurate and actionable, but the release it named was one #71 meant to
stop using.

Comment says the two must track each other, since nothing enforces it.
@Taure
Taure merged commit 06f9a42 into main Aug 11, 2026
6 checks passed
@Taure
Taure deleted the fix/elp-version-default branch August 11, 2026 14:57
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