Merge pull request #283 from node-gradle/upgrade-default-nodejs

Upgrade default node to 18 (newest LTS)

Author: deepy

.github/workflows/build-examples.yml

@@ -14,7 +14,7 @@ jobs:
       - name: Install Node.js
         uses: actions/setup-node@v3
         with:
-          node-version: 16
+          node-version: 18
       - name: Install yarn
         run: npm install -g yarn@1.22.17
       - name: Gradle Version

.github/workflows/build.yml

@@ -21,7 +21,7 @@ jobs:
       - name: Install Node.js
         uses: actions/setup-node@v3
         with:
-          node-version: 16
+          node-version: 18
       - name: Install yarn
         run: npm install -g yarn@1.22.17
       - name: Setup Gradle

.github/workflows/publish.yml

@@ -19,7 +19,7 @@ jobs:
       - name: Install Node.js
         uses: actions/setup-node@v3
         with:
-          node-version: 16
+          node-version: 18
       - name: Install yarn
         run: npm install -g yarn@1.22.17
       - name: Build

CHANGELOG.md

@@ -1,6 +1,7 @@
 # Changelog
 
 ## Version 6.x *(unreleased)*
+* Upgrade default Node to 18.17.1 and npm to 9.6.7
 
 ## Version 6.0.0 *(2023-08-15)*
 * Removed deprecated `nodeModulesDir` from `NodeExtension`

docs/usage.md

@@ -204,7 +204,7 @@ node {
 
     // Version of node to download and install (only used if download is true)
     // It will be unpacked in the workDir
-    version = "16.14.0"
+    version = "18.17.1"
 
     // Version of npm to use
     // If specified, installs it in the npmWorkDir

src/main/kotlin/com/github/gradle/node/NodeExtension.kt

@@ -186,12 +186,12 @@ open class NodeExtension(project: Project) {
         /**
          * Default version of Node to download if none is set
          */
-        const val DEFAULT_NODE_VERSION = "16.14.2"
+        const val DEFAULT_NODE_VERSION = "18.17.1"
 
         /**
          * Default version of npm to download if none is set
          */
-        const val DEFAULT_NPM_VERSION = "8.5.0"
+        const val DEFAULT_NPM_VERSION = "9.6.7"
 
         @JvmStatic
         operator fun get(project: Project): NodeExtension {

src/test/groovy/com/github/gradle/node/task/NodeTask_integTest.groovy

@@ -1,6 +1,7 @@
 package com.github.gradle.node.task
 
 import com.github.gradle.AbstractIntegTest
+import com.github.gradle.node.NodeExtension
 import org.gradle.testkit.runner.TaskOutcome
 import org.gradle.util.GradleVersion
 import org.junit.Assume
@@ -87,7 +88,7 @@ class NodeTask_integTest extends AbstractIntegTest {
 
         then:
         result9.task(":version").outcome == TaskOutcome.SUCCESS
-        result9.output.contains("Version: v16.14.2")
+        result9.output.contains("Version: v${DEFAULT_NODE_VERSION}")
 
         where:
         gv << GRADLE_VERSIONS_UNDER_TEST

src/test/groovy/com/github/gradle/node/yarn/task/YarnInstall_integTest.groovy

@@ -1,6 +1,8 @@
 package com.github.gradle.node.yarn.task
 
 import com.github.gradle.AbstractIntegTest
+import com.github.gradle.node.NodeExtension
+import com.github.gradle.node.NodePlugin
 import org.gradle.testkit.runner.TaskOutcome
 
 class YarnInstall_integTest extends AbstractIntegTest {
@@ -45,18 +47,18 @@ class YarnInstall_integTest extends AbstractIntegTest {
         given:
         gradleVersion = gv
 
-        writeBuild('''
+        writeBuild("""
             plugins {
                 id 'com.github.node-gradle.node'
             }
 
             node {
                 download = true
                 yarnWorkDir = file('build/yarn')
-                version = '16.14.2'
+                version = '${NodeExtension.DEFAULT_NODE_VERSION}'
                 npmVersion = '7.0.1'
             }
-        ''')
+        """)
         writeEmptyPackageJson()
 
         when:

src/test/resources/fixtures/kotlin/build.gradle.kts

@@ -16,7 +16,7 @@ plugins {
 }
 
 node {
-    version.set("16.14.0")
+    version.set("18.17.1")
     npmVersion.set("")
     yarnVersion.set("")
     npmInstallCommand.set("install")

src/test/resources/fixtures/node-allow-insecure-protocol/build.gradle

@@ -3,7 +3,7 @@ plugins {
 }
 
 node {
-    version = "16.14.2"
+    version = "18.17.1"
     distBaseUrl = "http://nodejs.org/dist/"
     download = true
     allowInsecureProtocol = true