Skip to content

request-info bot works after label is added.#36

Open
knrt10 wants to merge 3 commits into
behaviorbot:masterfrom
knrt10:label
Open

request-info bot works after label is added.#36
knrt10 wants to merge 3 commits into
behaviorbot:masterfrom
knrt10:label

Conversation

@knrt10

@knrt10 knrt10 commented Apr 18, 2018

Copy link
Copy Markdown
Contributor

As discussed in #30 , now bot will work when request-info is added as label when issue or PR is opened.

@update-docs

update-docs Bot commented Apr 18, 2018

Copy link
Copy Markdown

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.

@knrt10

knrt10 commented Apr 18, 2018

Copy link
Copy Markdown
Contributor Author

@hiimbex please verify the working so that I can update some tests according to that.

@knrt10 knrt10 changed the title Add enhancement to show request-info request-info bot works after label is added. Apr 18, 2018

@hiimbex hiimbex left a comment

Copy link
Copy Markdown
Member

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.

@knrt10

knrt10 commented Apr 18, 2018

Copy link
Copy Markdown
Contributor Author

@hiimbex I found simpler approach let me update that.

@knrt10 knrt10 changed the title request-info bot works after label is added. WIP request-info bot works after label is added. Apr 18, 2018
@knrt10 knrt10 changed the title WIP request-info bot works after label is added. request-info bot works after label is added. Apr 18, 2018
@knrt10

knrt10 commented Apr 18, 2018

Copy link
Copy Markdown
Contributor Author

@hiimbex please review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants