-
Notifications
You must be signed in to change notification settings - Fork 97
[readme] update required version of nodejs 20 or newer #5494
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
mihaibudiu
left a comment
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 am not sure it's sufficient, I had problems with some versions >= 20 in the past.
I have 25 for now.
|
I have Node 20 locally, so that should work; the issue must've been something else |
Without version 20 it won't compile. We use version 20 in our build image |
|
I just said that I am not sure that all versions above 20 will work. |
Got it, thanks. I will update readme to use exact version 20. |
|
In that case mention both 20 and 25 as "confirmed to work", I guess |
Need new styleText for the webconsole We use nodejs v20 in our build docker image
e9cc482 to
6abadfc
Compare
I think we better guide users to install the same version as we use in our build Docker image. |
nodejs v20 is needed for the styleText in the webconsole, we use version 20 in our build Docker image
Checklist
Breaking Changes?
Mark if you think the answer is yes for any of these components:
Describe Incompatible Changes
Add a few sentences describing the incompatible changes if any.