-
Notifications
You must be signed in to change notification settings - Fork 73
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
Crashes on node v12 #62
Comments
@dougmoscrop Is it possible to cut a release for this? I can't figure out how to include the fix in my deployment in package.json. |
Whoops! I will publish a release now. For future reference if you're in a pinch, you can use a github reference as a version in package.json: "serverless-plugin-split-stacks": "github:dougmoscrop/serverless-plugin-split-stacks#tag-or-sha" |
@dougmoscrop thanks very much! Good to know for the future. |
Published 1.7.3 with the fix |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In the below code we check for
error.message
to end with thecustomPath
.serverless-plugin-split-stacks/lib/migration-strategy/custom.js
Line 16 in 5332646
But starting with node.js v12, for
MODULE_NOT_FOUND
errors the error message also includes the require stack (nodejs/node#25690), which breaks the expected behavior that we want.The text was updated successfully, but these errors were encountered: