-
-
Notifications
You must be signed in to change notification settings - Fork 943
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
ci: use node 20 #2398
ci: use node 20 #2398
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## next #2398 +/- ##
=======================================
Coverage 99.61% 99.61%
=======================================
Files 2800 2800
Lines 252410 252410
Branches 1100 1098 -2
=======================================
Hits 251432 251432
Misses 951 951
Partials 27 27 |
This comment actions/runner#2033 (comment) potentially saved me multiple hours of debugging ❤️ |
This might solve the issue: |
This comment is a year old, I think the current solution is sufficient enough |
container: cypress/browsers:node18.12.0-chrome107 | ||
container: | ||
image: cypress/browsers:node-20.6.1-chrome-116.0.5845.187-1-ff-117.0-edge-116.0.1938.76-1 | ||
options: --user 1001 |
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.
I'm confused 🤔
This was literally the solution to the problem, now with that it works
Sorry async communication error. |
I will switch the CI requirements, when the PR gets the required approval amount to get merged |
Use the current node version
Look if it speeds up the CI by a little bit
Rough comparison: around 3min FASTER 🚀