-
Notifications
You must be signed in to change notification settings - Fork 42
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
request-info bot works after label is added. #36
base: master
Are you sure you want to change the base?
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would update some of our documentation based on your changes. |
@hiimbex please verify the working so that I can update some tests according to that. |
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.
Hi @knrt10 I'm very confused by your approach here. you're calling robot.on within a robot.on block and deleting a lot of tests that are very important. You need to look at the existing code for this bot so you don't repeat sections.
@hiimbex I found simpler approach let me update that. |
@hiimbex please review. |
As discussed in #30 , now bot will work when
request-info
is added as label when issue or PR is opened.