Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: Add support to build automatically npm dependencies #292

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
maintux wants to merge 9 commits into terraform-aws-modules:master from maintux:npm-support

Conversation

@maintux
Copy link

@maintux maintux commented Mar 26, 2022

  1. Add support to build automatically npm dependencies, with docker support.
  2. Change default docker image from lambci to official AWS SAM images

Description

As made for pip requirements, this PR adds support to build Nodejs dependencies automatically with or without docker.

Motivation and Context

This PR simplifies the process needed to build and deploy Nodejs functions and, thanks to docker, allows users to build functions without the need to have the interpreter installed locally

Breaking Changes

No breaking changes introduced

How Has This Been Tested?

  • [ x ] I have tested and validated these changes using one or more of the provided examples/* projects
  • [ x ] I have also tested the real deployment of Python and Nodejs functions with and without dependencies.

...ort. Change default docker image from lambci to offical AWS SAM images
@maintux maintux changed the title (削除) add support to build automatically npm dependencies (削除ここまで) (追記) feat: add support to build automatically npm dependencies (追記ここまで) Mar 26, 2022
@maintux maintux changed the title (削除) feat: add support to build automatically npm dependencies (削除ここまで) (追記) feat: Add support to build automatically npm dependencies (追記ここまで) Mar 26, 2022
Copy link
Member

@antonbabenko antonbabenko left a comment

Choose a reason for hiding this comment

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

Looks very good! Just a small addition - please mention this new feature in README somewhere and update code in examples to use it for real (this is going to be a confirmation that this PR does what it supposes). Thank you!

maintux reacted with thumbs up emoji
README.md Outdated
}, {
path = "src/nodejs14.x-app1",
npm_requirements = true,
nom_tmp_dir = "/tmp/dir/location"
Copy link
Member

@antonbabenko antonbabenko Mar 26, 2022

Choose a reason for hiding this comment

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

Suggested change
nom_tmp_dir = "/tmp/dir/location"
npm_tmp_dir = "/tmp/dir/location"

Copy link
Author

Choose a reason for hiding this comment

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

Done!

Copy link
Member

I have finished it in #293 because I didn't have permission to push to your fork.

Copy link
Member

This issue has been resolved in version 2.36.0 🎉

Copy link
Author

maintux commented Mar 26, 2022

Top! Thanks :)

antonbabenko reacted with hooray emoji

Copy link

github-actions bot commented Nov 8, 2022

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@antonbabenko antonbabenko antonbabenko requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /