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

Run CI on ubuntu-20.04 #200

Merged
merged 1 commit into from
Jan 23, 2023
Merged

Conversation

sato11
Copy link
Contributor

@sato11 sato11 commented Jan 23, 2023

Hi 👋 I came across and saw the CI not passing for some reason. This is the error message:

Requested Erlang/OTP version (21.3) not found in version list (should you be using option 'version-type': 'strict'?)

It seems this is because the ubuntu-latest is resolved ubuntu-22.04, which is not compatible with otp 21.3. Ref: https://github.com/erlef/setup-beam#compatibility-between-operating-system-and-erlangotp.

Downgrading os version might sound like a degradation, but I think we can excuse it since this is the way projects like plug run CI against older versions of otp.

This also updates actions/checkout from v2 to v3, consequently resolving deprecation warning that prompts using Nodejs v16. Ref: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/

I validated that this fixes it after running the workflow in my fork, if you want to take a look 🙂

This is because the ubuntu-latest is resolved ubuntu-22.04,
which is not compatible with otp 21.3 which this project supports.

This also updates actions/checkout from v2 to v3,
consequently resolving deprecation warning that prompts using Nodejs v16.
@Nitrino
Copy link
Member

Nitrino commented Jan 23, 2023

@sato11 Nice work!
I just didn't have time to fix it ❤️

@Nitrino Nitrino merged commit fb7cbfe into elixirmoney:master Jan 23, 2023
@sato11
Copy link
Contributor Author

sato11 commented Jan 24, 2023

Thanks 😉

@sato11 sato11 deleted the run-ci-on-ubuntu-20-04 branch January 24, 2023 02:17
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