-
Notifications
You must be signed in to change notification settings - Fork 923
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: Make probot receive
support complex Probot apps
#1714
fix: Make probot receive
support complex Probot apps
#1714
Conversation
The command `probot receive` fails to work when: 1. The Probot appFn is asynchronous 2. The Probot appFn requires `getRouter` The command `probot run` works fine though with the same app. This attempts to add support for these sort of Probot apps when calling `probot receive`.
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.
could you add a test to make sure we don't break this in future?
Thanks for reviewing @gr2m Do you have a recommendation on how I'd go about testing this? I didn't see an existing test file for |
probot receive
support complex Probot apps
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.
sorry on the wait!
🎉 This PR is included in version 12.2.8 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Oh my what a pleasant surprise when I came back from Holiday! 🎉 We're looking forward to using this in our project 😄 |
The command
probot receive
fails to work when:getRouter
The command
probot run
works fine though with the same Probot appFn.This attempts to add support for these sorts of Probot apps when calling
probot receive
.