-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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: add node engine check #7574
ci: add node engine check #7574
Conversation
Thanks for opening this pull request!
|
Codecov Report
@@ Coverage Diff @@
## master #7574 +/- ##
==========================================
- Coverage 93.93% 93.90% -0.03%
==========================================
Files 181 181
Lines 13281 13281
==========================================
- Hits 12475 12472 -3
- Misses 806 809 +3
Continue to review full report at Codecov.
|
🎉 This change has been released in version 5.0.0-beta.1 |
🎉 This change has been released in version 5.0.0 |
New Pull Request Checklist
Issue Description
Currently it is possible that a dependency of parse server requires a higher node engine than parse server itself. This has the potential to break parse server, while passing the CI.
Related issue: #n/a
Approach
Add node engine CI check
TODOs before merging