-
-
Notifications
You must be signed in to change notification settings - Fork 18
Docs: Update the minimum nodeJS version requirement #26
Conversation
Also: no need to specify the /en/ in the nodejs.org URL, because the web server sets it automatically depending on the browser language.
Hi @0uep!, thanks for the Pull Request The first commit message isn't properly formatted. We ask that you update the message to match this format, as we use it to generate changelogs and automate releases.
Read more about contributing to ESLint here |
Hi @0uep!, thanks for the Pull Request The first commit message isn't properly formatted. We ask that you update the message to match this format, as we use it to generate changelogs and automate releases.
Read more about contributing to ESLint here |
Hi @0uep!, thanks for the Pull Request The first commit message isn't properly formatted. We ask that you update the message to match this format, as we use it to generate changelogs and automate releases.
Read more about contributing to ESLint here |
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.
It seems we forgot to update README, thanks for fixing it!
Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com>
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.
LGTM, thanks!
Can you please sign our CLA so we could merge this?
Also: no need to specify the /en/ in the nodejs.org URL, because the web server sets it automatically depending on the browser language.