-
Notifications
You must be signed in to change notification settings - Fork 53
node_modules not included in layers #163
Comments
Same here. The latest version generates a layer without node_modules, it contains just package-lock.json file. |
Same here. I downgrade version and its working fine... |
Same here, downgraded to
|
Im too. I have the same problem. I downgraded too to 2.7.0 and still showing error on deploy =(
@agutoli help us please. The project stop to upload beacause this problem. Sorry and thanks ^^ Updated: @rsi-mrobinson i changed the "serverless-layers" from devDependencies to dependencies and set the version to "2.7", yes without the third number and worked to me! \O/ |
If anyone is facing issues even after downgrading to
|
Related to #166 |
I made localized modifications to the source code. This is not safe, but at least it will not prevent the normal deployment of the service. |
Issue Description:
After upgrading to version
v2.8.5
ofserverless-layers
, the plugin fails to include thenode_modules
directory in its generated layers.Steps to Reproduce:
serverless-layers
tov2.8.5
.Expected Behavior:
The layers should contain the
node_modules
directory, including all dependencies required by the specified packages.Actual Behavior:
Upon inspection of the deployed layers, we can see that the
node_modules
directory is missing.Workaround:
Reverting to version
v2.7.0
ofserverless-layers
resolves the issue, as thenode_modules
directory is correctly included in the generated layers.Environment:
3.38.0
18.17.1
9.6.7
Screenshot:
![example](https://private-user-images.githubusercontent.com/70632308/311533804-d1a53d21-4aee-4cb2-bc0b-5cda14444754.jpeg?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTkxMDk3MTQsIm5iZiI6MTcxOTEwOTQxNCwicGF0aCI6Ii83MDYzMjMwOC8zMTE1MzM4MDQtZDFhNTNkMjEtNGFlZS00Y2IyLWJjMGItNWNkYTE0NDQ0NzU0LmpwZWc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNjIzJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDYyM1QwMjIzMzRaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT04NDhjYWViOTJjNjU4YmZmYjViOWUyNzU5ZTE3OWEwMzUwY2MwZGJhNGViMTM1YjQyZGQzYzUwOTg1M2U0MWRlJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.p1O0rkX4I0GvocvnSu9ooI0nrAYHgFQC8pt6ZUhZp1w)
Here you can observe that the total layer size is much less than it should be.
The text was updated successfully, but these errors were encountered: