Skip to content

Commit

Permalink
Merge branch 'master' into fix-runtimes
Browse files Browse the repository at this point in the history
  • Loading branch information
antonbabenko authored Oct 27, 2023
2 parents f878a5b + e2500c0 commit 55f346a
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 5 deletions.
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,13 @@

All notable changes to this project will be documented in this file.

### [6.0.1](https://github.com/terraform-aws-modules/terraform-aws-lambda/compare/v6.0.0...v6.0.1) (2023-10-05)


### Bug Fixes

* Fixed npm install on Windows without having to use wsl ([#502](https://github.com/terraform-aws-modules/terraform-aws-lambda/issues/502)) ([ffa56e8](https://github.com/terraform-aws-modules/terraform-aws-lambda/commit/ffa56e896d7c5e5c8cbc851f0c453b70e4ec100f))

## [6.0.0](https://github.com/terraform-aws-modules/terraform-aws-lambda/compare/v5.3.0...v6.0.0) (2023-08-09)


Expand Down
3 changes: 1 addition & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -559,8 +559,7 @@ module "lambda_function_existing_package_from_remote_url" {

## <a name="sam_cli_integration"></a> How to use AWS SAM CLI to test Lambda Function?
[AWS SAM CLI](https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/serverless-sam-cli-command-reference.html) is an open source tool that help the developers to initiate, build, test, and deploy serverless
applications. Currently, SAM CLI tool only supports CFN applications, but SAM CLI team is working on a feature to extend the testing capabilities to support terraform applications (check this [Github issue](https://github.com/aws/aws-sam-cli/issues/3154)
to be updated about the incoming releases, and features included in each release for the Terraform support feature).
applications. SAM CLI tool [supports Terraform applications](https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/what-is-terraform-support.html).

SAM CLI provides two ways of testing: local testing and testing on-cloud (Accelerate).

Expand Down
10 changes: 7 additions & 3 deletions package.py
Original file line number Diff line number Diff line change
Expand Up @@ -1240,12 +1240,16 @@ def install_npm_requirements(query, requirements_file, tmp_dir):
shutil.copyfile(requirements_file, target_file)

subproc_env = None
if not docker and OSX:
subproc_env = os.environ.copy()
npm_exec = 'npm'
if not docker:
if WINDOWS:
npm_exec = 'npm.cmd'
elif OSX:
subproc_env = os.environ.copy()

# Install dependencies into the temporary directory.
with cd(temp_dir):
npm_command = ['npm', 'install']
npm_command = [npm_exec, 'install']
if docker:
with_ssh_agent = docker.with_ssh_agent
chown_mask = '{}:{}'.format(os.getuid(), os.getgid())
Expand Down

0 comments on commit 55f346a

Please sign in to comment.