fix: When npm_requirements is included in a source_path claim it is not running npm install

terraform-aws-modules / terraform-aws-lambda

Terraform module, which takes care of a lot of AWS Lambda/serverless tasks (build dependencies, packages, updates, deployments) in countless combinations 🇺🇦

https://registry.terraform.io/modules/terraform-aws-modules/lambda/aws

Apache License 2.0

886 stars 656 forks source link

fix: When npm_requirements is included in a source_path claim it is not running npm install #507

Closed phildier closed 6 months ago

phildier commented 8 months ago

Description

This is a fix for this issue https://github.com/terraform-aws-modules/terraform-aws-lambda/issues/506

Motivation and Context

See context in issue https://github.com/terraform-aws-modules/terraform-aws-lambda/issues/506

Breaking Changes

No breaking changes

How Has This Been Tested?

I have tested this with a nodejs lambda in my environment

github-actions[bot] commented 6 months ago

This PR has been automatically marked as stale because it has been open 30 days with no activity. Remove stale label or comment or this PR will be closed in 10 days

github-actions[bot] commented 6 months ago

This PR was automatically closed because of stale in 10 days

github-actions[bot] commented 5 months ago

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.