test: Remove support for Node 18

edx · Oct 10, 2024 · 3a58401 · 3a58401
1 parent b180478
commit 3a58401
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -13,7 +13,6 @@ jobs:
         python-version: ["3.8", "3.12"]
         os: [ubuntu-20.04]
         toxenv: [django42]
-        node: [18, 20]
     env:
       DATA_API_VERSION: "latest"
     steps:

diff --git a/Dockerfile b/Dockerfile
@@ -63,8 +63,8 @@ WORKDIR ${INSIGHTS_CODE_DIR}/
 # insights service config commands below
 RUN pip install  --no-cache-dir -r ${INSIGHTS_CODE_DIR}/requirements/production.txt 
 
-RUN nodeenv ${INSIGHTS_NODEENV_DIR} --node=18.20.2 --prebuilt \
-  && npm install -g npm@10.5.x
+RUN nodeenv ${INSIGHTS_NODEENV_DIR} --node=20.15.1 --prebuilt \
+  && npm install -g npm@10.7.x
 
 # Tried to cache the dependencies by copying related files after the npm install step but npm post install fails in that case.
 COPY . ${INSIGHTS_CODE_DIR}/