-
Notifications
You must be signed in to change notification settings - Fork 293
Automatically add to sys.path #1
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
Comments
Agree. This plugin is great in theory but having to modify every file in my code to support it is undesirable. |
@dschep why not put the python dependencies in the project root where they would normally be? |
@tonycapone that just seems too messy, and there'd be no way to easily git ignore all of them. But yeah. that certainly would take care of it. |
Also, it would be incompatible with #4 😢 |
@dschep Yeah that's true, although that messiness is a problem in all Python Aws Lambda projects. FWIW here's my workaround that mitigated most of the pain: Put all source code into a local package. Put the Lambda handler file in the project root, and import the package from the handler file. Then you only have to put |
allow per-function requirements plus base layer
Maybe there's a way to avoid having to add the
import requirements
to your Python files...The text was updated successfully, but these errors were encountered: