Skip to content
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

feat(serverless-nestjs): added example implementation of nestjs with … #394

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

qutek
Copy link
Contributor

@qutek qutek commented Feb 22, 2024

The article "Container Image Support for AWS Lambda" on the Serverless Framework blog introduces the ability to use container images with AWS Lambda, allowing developers to have full control over the execution environment and access to custom runtimes and libraries.

This PR adds a basic example of using the aws-lambda-web-adapter package with the NestJS framework, and the Serverless Framework version 3.

@bnusunny
Copy link
Contributor

Could you move this under "examples/sls" directory? I want to separate examples using different IaC tools.

@qutek
Copy link
Contributor Author

qutek commented Apr 3, 2024

@bnusunny sorry, I just realized your comment.

so the files need to be located under /examples/sls or /examples/sls/serverless-nestjs?

@bnusunny
Copy link
Contributor

bnusunny commented Apr 4, 2024

Please put this example into /examples/sls/nextjs directory.

In addition, you need to redirect nextjs cache directory under /tmp. This enables people to use nextjs cache. Take a look at this and this files.

@qutek
Copy link
Contributor Author

qutek commented Apr 4, 2024

@bnusunny okay,

this example is actually for nestjs (not nextjs), sorry if it is confusing

@bnusunny bnusunny merged commit 8061cf1 into awslabs:main Apr 5, 2024
23 checks passed
@bnusunny
Copy link
Contributor

bnusunny commented Apr 5, 2024

Thanks for the contribution! @qutek

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