-
Notifications
You must be signed in to change notification settings - Fork 114
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
Feature: Option worker definition optionally as string reference. #417
base: master
Are you sure you want to change the base?
Conversation
Thanks! I wonder if this could work instead: functions:
foo:
handler: worker.handler
constructs:
emails:
type: queue
worker: foo that would be simpler. |
Does Serverless support varied types? |
This pattern is very common in serverless.yml (string or object value). |
0812436
to
3259f65
Compare
I think I figured out how to specify the function and updated the PR to make worker optionally as a string |
Can you document this new feature here: https://github.com/getlift/lift/blob/master/docs/queue.md#worker Also I think it might be worth rebasing after we merge #416 so that we get a clear picture of the CI. |
fixing typo Co-authored-by: Matthieu Napoli <[email protected]>
47f363b
to
706d2ab
Compare
@mnapoli I added some documentation and rebased the branch. |
@mnapoli is there something else I can do? |
This is a stab at a reference defining the worker reference following the discussion in #415