-
Notifications
You must be signed in to change notification settings - Fork 5
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
[PRMP-620] bump up nodejs and packages version #83
Conversation
joefong-nhs
commented
Jul 26, 2024
- bump up nodejs version to 22.5
- bump up packages version to address audit alerts
- fix an issue caused by using newer version of Axios (header silently ignored if the value is boolean false, see Axios removes false values from headers axios/axios#6013 (comment))
…broken test due to axios version up
…stead of boolean value (as Axios will silently neglect boolean false as header value)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of these changes are looking good and match those we've seen on ehr-out
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of these changes are looking good and match those we've seen on ehr-out
.
Just one comment to review.
Report for environment: pre-prodTerraform Format and Style 🖌
|
Report for environment: devTerraform Format and Style 🖌
|
Report for environment: prodTerraform Format and Style 🖌
|
Report for environment: testTerraform Format and Style 🖌
|