-
Notifications
You must be signed in to change notification settings - Fork 159
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: issue fixed to generate the correct function.json for event hub and service bus queue and topic triggers #228
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change fixed:
to fix:
before committing. There should be some kind of unit test accompanying each change. It would probably be a test within the packageService
or bindings
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - Added 1 minor comment.
@njm3754 - Please make sure you associate the github issues with this fix. |
Resolves issue #57 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…and service bus queue and topic triggers (#228) * fixed: issue fixed to create the right function.json for event hub trigger * fix: wrong bindings issue for servicebus topic and queue added * Test cases added * Test cases added * eventhubame to eventHubName
No description provided.