CDCgov / prime-simplereport

SimpleReport is a fast, free, and easy way for COVID-19 testing facilities to report results to public health departments.
https://simplereport.gov
Creative Commons Zero v1.0 Universal
58 stars 53 forks source link

Set node version to 23.1 #8280

Closed emyl3 closed 1 week ago

emyl3 commented 1 week ago

FRONTEND PULL REQUEST

Related Issue

Our frontend build job was initially passing following the Node 23 upgrade. Recently, the same job began to fail.

@mehansen found that version 23.2.x of NodeJS released on November 11, 2024 is breaking our frontend builds

Changes Proposed

Additional Information

Previous comment

⚠️ I didn't change the image in our frontend Dockerfile because it looks like that image uses node 23.1.0. Let me know if you prefer I choose a different image though 🤔

Updated to a Docker image with node 23.1.0 set

See other reported issues:

Internal convo here

Testing

Screenshots / Demos

mehansen commented 1 week ago

do we know how often the Docker images are updated and how that impacts us? I assume someone would update it to use 23.2 soon enough. if that happens will our frontend start failing to build?

tagging @shanice-skylight in case she has more insight

emyl3 commented 1 week ago

do we know how often the Docker images are updated and how that impacts us? I assume someone would update it to use 23.2 soon enough. if that happens will our frontend start failing to build?

tagging @shanice-skylight in case she has more insight

During pairing @mehansen found this image that sets it to node 23.1 so I'll update with this one for now!

sonarcloud[bot] commented 1 week ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

mehansen commented 1 week ago

smoke tested in dev6. checks pass, so looks good to me