-
Notifications
You must be signed in to change notification settings - Fork 13
Update GitHub Actions tool versions and keep them consistent #210
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
Conversation
you are right about postinstall
|
... and just when continuing upgrading to major version 4, they released five! |
I was quite overwhelmed when they released v3 with amount of breaking changes :D |
@paazmaya some changes revealed old issue that I completely forgot Visual-Regression-Tracker/Visual-Regression-Tracker#137 that revealed the gap I used to check manually after docker images are build so I added acceptance step to test docker image is build and containers are up and running |
af42c0a
to
9fc2dda
Compare
Lets see if downgrading Ubuntu will make the difference. I dont want to downgrade Node.js as I like to follow https://github.com/nodejs/release#release-schedule so that the "maintenance" version is the lowest. |
Looks like Ubuntu version was the case for unit tests failing due to missing Prisma postinstall, now CI is failing later...
How about getting #206 thru first and then see if this problem repeats or gets fixed? |
I think we could make it working at least with note 14 together with acceptance tests for docker images |
* Add acceptance tests * Use a specific Postgres version --------- Co-authored-by: Juga Paazmaya <paazmaya@yahoo.com>
Replacing #208 to have cleaner history.
Well this keeps failing, most likely
postinstall
of prisma is not triggered or something, but I will look again once #206 is done.