Skip to content

Commit

Permalink
WIP: make npm an explicit dep of node images (chainguard-images#2297)
Browse files Browse the repository at this point in the history
Signed-off-by: Jason Hall <[email protected]>
  • Loading branch information
imjasonh authored Mar 28, 2024
1 parent ae0474f commit 65fcd93
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion images/node-lts/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ variable "target_repository" {

module "latest-config" {
source = "../node/config"
extra_packages = ["nodejs-lts"]
extra_packages = ["nodejs-lts", "npm"]
}

module "latest" {
Expand Down
2 changes: 1 addition & 1 deletion images/node/config/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ terraform {

variable "extra_packages" {
description = "The additional packages to install (e.g. nodejs, nodejs-lts, nodejs-18)."
default = ["nodejs"]
default = ["nodejs", "npm"]
}

data "apko_config" "this" {
Expand Down
1 change: 1 addition & 0 deletions images/pulumi/config/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ output "config" {
# for Pulumi Node.js support
"pulumi-language-nodejs",
"nodejs",
"npm",
"nghttp2",
# for Pulumi Java support
"pulumi-language-java",
Expand Down

0 comments on commit 65fcd93

Please sign in to comment.