-
Notifications
You must be signed in to change notification settings - Fork 39
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
Add test for alpine 3.20.2 with chrome 128 #330
base: main
Are you sure you want to change the base?
Conversation
@@ -1,3 +1,3 @@ | |||
# locked to the versions we use in the lint CI job | |||
elixir 1.14.5 | |||
erlang 25.3.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This version is not working properly on a Mac without a lot of hassle.
c6c7632
to
8ab58e5
Compare
8ab58e5
to
416c88b
Compare
@@ -31,7 +31,7 @@ jobs: | |||
packages: write | |||
strategy: | |||
matrix: | |||
dockerfile: ['elixir-1-11', 'debian-buster', 'alpine-3-17-3', 'alpine-3-18-4'] | |||
dockerfile: ['elixir-1-11', 'debian-buster', 'alpine-3-17-3', 'alpine-3-18-4', 'alpine-3-20-2'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already ran it once on this branch :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤷 Looks good, but still fails on my machine (ie. I've built this locally and ran the tests)
Can we try to reproduce this on a CI run somehow? |
Not sure how, I mean we already run it inside this very container, on comparable x86-64 hardware on Linux. Maybe kernel version or some config issue? Not sure 🤷 Should we still merge this? |
No description provided.