Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

Commit

Permalink
refactor: upgrade to node 20
Browse files Browse the repository at this point in the history
refactor: upgrade to node 20
  • Loading branch information
giulianok authored Mar 12, 2024
1 parent 4374848 commit adbff4e
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 20 deletions.
2 changes: 1 addition & 1 deletion .nvmrc
Original file line number Diff line number Diff line change
@@ -1 +1 @@
18.17.1
20.11.1
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
ARG VERSION=lts
ARG VERSION=20
# Use the pre-baked fat node image only in the builder
# which includes build utils preinstalled (e.g. gcc, make, etc).
# This will result in faster and reliable One App docker image
# builds as we do not have to run apk installs for alpine.
FROM node:$VERSION as builder
WORKDIR /opt/build
RUN npm install -g npm@9.6.7 --registry=https://registry.npmjs.org
RUN npm install -g npm@10.2.3 --registry=https://registry.npmjs.org
COPY --chown=node:node ./ /opt/build
# npm ci does not run postinstall with root account
RUN NODE_ENV=development npm ci --build-from-source
Expand Down
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

17 changes: 3 additions & 14 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@
"description": "One Amex SPA technology stack.",
"main": "index.js",
"engines": {
"node": ">=18 <=20",
"npm": ">=8"
"node": ">=18",
"npm": ">=9"
},
"scripts": {
"preinstall": "npx check-engines@1",
Expand Down Expand Up @@ -55,18 +55,7 @@
"bugs": {
"url": "https://github.com/americanexpress/one-app/issues"
},
"contributors": [
"Andres Escobar <Andres.Escobar@aexp.com> (https://github.com/anescobar1991)",
"James Singleton <James.Singleton1@aexp.com> (https://github.com/JamesSingleton)",
"Jamie King <Jamie.King@aexp.com> (https://github.com/10xLaCroixDrinker)",
"Jonathan Adshead <Jonathan.Adshead@aexp.com> (https://github.com/JAdshead)",
"Michael Tobia <Michael.M.Tobia@aexp.com> (https://github.com/Francois-Esquire)",
"Michael Tomcal <Michael.A.Tomcal@aexp.com> (https://github.com/mtomcal)",
"Stephanie Coates <Stephanie.Coates1@aexp.com> (https://github.com/stephaniecoates)",
"Nelly Kiboi <Nelly.J.Kiboi@aexp.com> (https://github.com/nellyk)",
"Nickolas Oliver <nickolas.oliver@aexp.com> (https://github.com/PixnBits)",
"Ruben Casas <ruben.casas@aexp.com> (https://github.com/infoxicator)"
],
"author": "One App Team <oneamex.careers@aexp.com>",
"license": "Apache-2.0",
"keywords": [
"spa",
Expand Down
2 changes: 1 addition & 1 deletion prod-sample/api/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
ARG VERSION=lts
ARG VERSION=20

FROM node:$VERSION as builder
MAINTAINER One App Team
Expand Down

0 comments on commit adbff4e

Please sign in to comment.