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

Additional Lambda Handler detection for setting module.exports to an object #143

Open
awschristou opened this issue Oct 26, 2018 · 0 comments
Labels
feature-request New feature or enhancement. May require GitHub community feedback. language:node debugging NodeJS / javascript / typescript lambdas using SAM local

Comments

@awschristou
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Extend the javascript Lambda Handler candidate detection from Issue #140 PR #142 to support the following scenarios:

const myExports = {
    foo: // ...
    bar: // ...
}

[module.]exports = myExports

// and

[module.]exports = {
    foo: // ...
    bar: // ...
}

Describe the solution you'd like

Where foo and bar are valid candidates for Lambda function handlers, they are detected with the rest of the candidates.

Describe alternatives you've considered

Additional context

@awschristou awschristou added help wanted language:node debugging NodeJS / javascript / typescript lambdas using SAM local labels Oct 26, 2018
@awschristou awschristou removed this from the v1.0 release to marketplace milestone Dec 13, 2018
@mpiroc mpiroc added feature-request New feature or enhancement. May require GitHub community feedback. and removed technical-debt labels May 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature or enhancement. May require GitHub community feedback. language:node debugging NodeJS / javascript / typescript lambdas using SAM local
Projects
None yet
Development

No branches or pull requests

2 participants