Repository navigation
chore: main to dev merge - #725
Merged
Merged
Conversation
chore: Dev to Main
chore: Dev to Main
fix: Upgrade Node.js version
Coverage Report •
|
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues remain for the Node.js 24 configuration updates.
No open issues 🎉
What changed in this PR
This main-to-dev merge updates Node.js configuration across the web application, workflow service, and development containers to target Node.js 24.
Changes:
- Updates production build versions and the web application's Node.js engine requirement.
- Aligns development containers with Node.js 24 and switches web devcontainers to newer Microsoft image paths.
- Updates a commented-out CI example to Node.js 24; active CI behavior is unchanged.
| File | Description |
|---|---|
src/ContentProcessorWorkflow/Dockerfile |
Changes the Node.js download version to 24.22.1. |
src/ContentProcessorWorkflow/.devcontainer/devcontainer.json |
Selects Node.js 24 for workflow development. |
src/ContentProcessorWeb/package.json |
Requires Node.js >=24.22.1 <25. |
src/ContentProcessorWeb/Dockerfile |
Changes the builder image to node:24.22.1-bookworm. |
src/ContentProcessorWeb/.devcontainer/Dockerfile |
Uses the newer TypeScript devcontainer image path and 24-bookworm variant. |
src/ContentProcessorWeb/.devcontainer/devcontainer.json |
Passes 24-bookworm as the build variant. |
src/ContentProcessorWeb/.devcontainer/base.Dockerfile |
Uses the newer JavaScript devcontainer image path and 24-bookworm variant. |
.github/workflows/test.yml |
Updates only the commented Node.js setup example. |
.devcontainer/Dockerfile |
Installs Node.js from the NodeSource 24.x channel. |
Review effort: Balanced
Roopan-Microsoft
approved these changes
Nov 2, 2026
Prajwal-Microsoft
approved these changes
Nov 2, 2026
This branch had an error being deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Merging main to dev
Does this introduce a breaking change?
Golden Path Validation
Deployment Validation
What to Check
Verify that the following are valid
Other Information