FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bump nodejs version in Dockerfile by cnvergence · Pull Request #521 · kbind-dev/kbind · GitHub

bump nodejs version in Dockerfile - #521

Merged
cnvergence merged 1 commit into
kbind-dev:mainfrom
cnvergence:fix/dockerfile-node-version
Apr 9, 2026
Merged

bump nodejs version in Dockerfile#521
cnvergence merged 1 commit into
kbind-dev:mainfrom
cnvergence:fix/dockerfile-node-version

Conversation

cnvergence commented Apr 8, 2026
edited
Loading

Copy link
Copy Markdown
Member

Summary

After updating web packages versions, we also need to bump node js version in Dockerfile.
Also, removes redundant npm install + npm ci --no-optional, replaced with just npm ci — The old setup ran npm install then immediately npm ci --no-optional (which nukes node_modules and reinstalls from lockfile anyway, making the first install pointless).

What Type of PR Is This?

Related Issue(s)

Fixes #

Release Notes

NONE

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
cnvergence requested a review from a team as a code owner April 8, 2026 15:14

xrstf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

/approve

cnvergence merged commit fce0e6c into kbind-dev:main Apr 9, 2026
7 checks passed
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL