-
-
Notifications
You must be signed in to change notification settings - Fork 793
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
fix: package exports for usage in plugins #1552
Conversation
name export of lambda for usage in plugins like sqs-offline or eventbridge-offline
@rubenkaiser do you know when are we expecting this PR to be merged? I am blocked on using eventbridge-offline with latest version of serverless-offline. Thanks! |
I have no control over that. Hopefully it will be merged soon after which I will release a new version of serverless-offline-eventbridge and submit a PR for an update in offline-sns. |
thank you for the PR @rubenkaiser ! you mentioned here: rubenkaiser/serverless-offline-eventBridge#60 (comment) that other plugins (e.g. sqs?) are importing "several things". I haven't found time to look into it yet, but do you know by any chance what else is needed - other than the Lambda class? |
I was just digging through the I wonder if we could (or even should?) do something similar mentioned here: https://nodejs.org/dist/latest-v18.x/docs/api/packages.html#extensions-in-subpaths e.g.
I'm contemplating a bit and I believe I'm leaning more towards that approach. what are your opinions on this matter? |
In my test setup I first tried this approach as it seems to be the nicest solution. For some reason I kept getting errors importing the submodules. Not sure why. I will have another look somewhere next week. |
@rubenkaiser could you have a look if this would work for you: #1552 |
@rubenkaiser sorry, linked to your own PR 🤦 😄 I meant this one: #1561 |
super seeded by: #1561 |
named export of lambda for usage in plugins like sqs-offline or eventbridge-offline