Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to Node v18 #209

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Update to Node v18 #209

wants to merge 8 commits into from

Conversation

ChisomOguibe11
Copy link
Contributor

What?

Update project to Node version 18. see jira ticket #UKVIET-55.
https://collaboration.homeoffice.gov.uk/jira/browse/UKVIET-55

Why?

Part of Epic to upgrade all HOF services to Node v18. see jira ticket #HOFF-312
https://collaboration.homeoffice.gov.uk/jira/browse/HOFF-312

How?

  • Updated node engine inside package.json from 14.15.0 to 18.12.1
  • Updated main docker image to node:18-alpine
  • Changed all references to node:14 in drone.yml to node:18

Testing?

  • All unit tests + acceptance tests pass
  • Requires testing from QAT team

Screenshots (optional)

Anything Else?

- Change Dockerfile base images from node14 to node18
- Change node engine in package.json to 18.12.1
- Change node:14 references .drone.yaml to node:18
- renew expiry dates for all snyk issues that expired in 2021
@ChisomOguibe11 ChisomOguibe11 marked this pull request as ready for review March 31, 2023 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants