8000 Fix for `rebar3` `nightly`/`latest` by paulo-ferraz-oliveira · Pull Request #331 · erlef/setup-beam · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix for rebar3 nightly/latest #331

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

Merged
merged 5 commits into from
May 25, 2025
Merged

Fix for rebar3 nightly/latest #331

merged 5 commits into from
May 25, 2025

Conversation

paulo-ferraz-oliveira
Copy link
Collaborator

Description

Compatibility with Erlang/OTP versions lower than 26 is not guaranteed, so we try to move the versions forward / adapt them, when warranted to prevent issues around e.g. rebar3 releases like the recent 3.25 one.

elixir-version: '1.16'
rebar3-version: '3'
rebar3-version: '3.25'
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We pin it, so it doesn't catch us by surprise in the (near?) future.

@@ -145,11 +142,11 @@ jobs:
fail-fast: false
matrix:
combo:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We benefit from latest, consistently, and thus this should break much less in the future.

@@ -128,11 +125,11 @@ jobs:
fail-fast: false
matrix:
combo:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We benefit from latest, consistently, and thus this should break much less in the future.

These tests seem to present issues in rebar3
(taking forever to compile) or verl, so I'm
avoiding nightly, for now
Not exactly sure what's wrong with this one, but
I don't think (given all the other successes in the
actions' tests) this is action-specific
@paulo-ferraz-oliveira paulo-ferraz-oliveira marked this pull request as ready for review May 25, 2025 11:35
@paulo-ferraz-oliveira
Copy link
Collaborator Author

I wrote this one to unlock #330 as it's failing since 3.25 came out, but due to our tests, not the proposed changes. Feel free to merge if you approve.

Copy link
Member
@starbelly starbelly left a comment

Choose a reason for hiding this comment

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

LGTM

@starbelly starbelly merged commit f28fd8e into main May 25, 2025
50 checks passed
@starbelly starbelly deleted the fix/rebar3 branch May 25, 2025 15:42
@paulo-ferraz-oliveira
Copy link
Collaborator Author

This change was released in v1.19.0, also tagged v1.19 and v1 (moved).

@wojtekmach
Copy link
Collaborator

👍

Comment on lines -47 to -49
- otp-version: '26.1'
rebar3-version: 'nightly'
os: 'windows-2019'
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

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.

3 participants
0