Unverified Commit 3442a4e8 authored by Kaniska's avatar Kaniska Committed by GitHub

[node] - Node 18 EOL changes (#1349)

* Node 18 EOL changes

* Fixing failing tests

* Updating based on review comments

---------
Co-authored-by: 's avatarÁlvaro Rausell Guiard <33221237+AlvaroRausell@users.noreply.github.com>
parent 600cda6c
{ {
"id": "node", "id": "node",
"version": "1.6.2", "version": "1.6.3",
"name": "Node.js (via nvm), yarn and pnpm", "name": "Node.js (via nvm), yarn and pnpm",
"documentationURL": "https://github.com/devcontainers/features/tree/main/src/node", "documentationURL": "https://github.com/devcontainers/features/tree/main/src/node",
"description": "Installs Node.js, nvm, yarn, pnpm, and needed dependencies.", "description": "Installs Node.js, nvm, yarn, pnpm, and needed dependencies.",
...@@ -11,9 +11,8 @@ ...@@ -11,9 +11,8 @@
"lts", "lts",
"latest", "latest",
"none", "none",
"18", "22",
"16", "20"
"14"
], ],
"default": "lts", "default": "lts",
"description": "Select or enter a Node.js version to install" "description": "Select or enter a Node.js version to install"
......
...@@ -5,13 +5,12 @@ set -e ...@@ -5,13 +5,12 @@ set -e
# Optional: Import test library # Optional: Import test library
source dev-container-features-test-lib source dev-container-features-test-lib
# 'lts' is now some version of node 20... # 'lts' is now some version of node 22...
check "version_on_path" node -v | grep 20 check "version_on_path" node -v | grep 22
check "pnpm" bash -c "pnpm -v | grep 8.8.0" check "pnpm" bash -c "pnpm -v | grep 8.8.0"
check "v20_installed" ls -1 /usr/local/share/nvm/versions/node | grep 20 check "v20_installed" ls -1 /usr/local/share/nvm/versions/node | grep 20.19.1
check "v14_installed" ls -1 /usr/local/share/nvm/versions/node | grep 14.19.3 check "v19_installed" ls -1 /usr/local/share/nvm/versions/node | grep 19.9.0
check "v17_installed" ls -1 /usr/local/share/nvm/versions/node | grep 17.9.1
# Report result # Report result
......
...@@ -5,13 +5,13 @@ set -e ...@@ -5,13 +5,13 @@ set -e
# Optional: Import test library # Optional: Import test library
source dev-container-features-test-lib source dev-container-features-test-lib
# 'lts' is now some version of node 20... # 'lts' is now some version of node 22...
check "version_on_path" node -v | grep 20 check "version_on_path" node -v | grep 22
check "pnpm" bash -c "pnpm -v | grep 6.16.0" check "pnpm" bash -c "pnpm -v | grep 6.16.0"
check "v20_installed" ls -1 /usr/local/share/nvm/versions/node | grep 20 check "v20_installed" ls -1 /usr/local/share/nvm/versions/node | grep 20
check "v14_installed" ls -1 /usr/local/share/nvm/versions/node | grep 14.19.3 check "v19_installed" ls -1 /usr/local/share/nvm/versions/node | grep 19.9.0
check "v17_installed" ls -1 /usr/local/share/nvm/versions/node | grep 17.9.1 check "v20_installed" ls -1 /usr/local/share/nvm/versions/node | grep 20.19.1
# Report result # Report result
......
#!/bin/bash
set -e
# Optional: Import test library
source dev-container-features-test-lib
# Definition specific tests
check "version" bash -c "node --version | grep 22"
check "pnpm" bash -c "pnpm -v | grep 8.8.0"
check "nvm" bash -c ". /usr/local/share/nvm/nvm.sh && nvm install 10"
# Report result
reportResults
...@@ -12,7 +12,7 @@ ...@@ -12,7 +12,7 @@
"features": { "features": {
"node": { "node": {
"version": "lts", "version": "lts",
"additionalVersions": "v17.9.1,v14.19.3", "additionalVersions": "v20.19.1,v19.9.0",
"pnpmVersion": "8.8.0" "pnpmVersion": "8.8.0"
} }
} }
...@@ -22,7 +22,7 @@ ...@@ -22,7 +22,7 @@
"features": { "features": {
"node": { "node": {
"version": "lts", "version": "lts",
"additionalVersions": "v17.9.1,v14.19.3", "additionalVersions": "v20.19.1,v19.9.0",
"pnpmVersion": "6.16.0" "pnpmVersion": "6.16.0"
} }
} }
...@@ -77,11 +77,11 @@ ...@@ -77,11 +77,11 @@
} }
} }
}, },
"install_node_16_on_bionic": { "install_node_22_on_jammy": {
"image": "mcr.microsoft.com/devcontainers/base:ubuntu-18.04", "image": "mcr.microsoft.com/devcontainers/base:ubuntu-22.04",
"features": { "features": {
"node": { "node": {
"version": "16", "version": "22",
"pnpmVersion":"8.8.0" "pnpmVersion":"8.8.0"
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment