Skip to content
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

[BUG] Bot should ignore / commands that are embedded in code #66

Open
asmacdo opened this issue Jul 13, 2022 · 4 comments
Open

[BUG] Bot should ignore / commands that are embedded in code #66

asmacdo opened this issue Jul 13, 2022 · 4 comments
Assignees
Labels

Comments

@asmacdo
Copy link

asmacdo commented Jul 13, 2022

Describe the bug
When putting a / command in a code segment, the bot should not follow the command.

/approve

To Reproduce
I just did! The bot should reply with an erroneous comment, see below :)

@asmacdo
Copy link
Author

asmacdo commented Jul 13, 2022

/lgtm

@github-actions
Copy link
Contributor

Cannot apply the lgtm label because Error: asmacdo is not a org member or collaborator

@jpmcb
Copy link
Owner

jpmcb commented Jul 16, 2022

Hmmmm interesting! Thanks for this feedback - I think this is something we could do easily!

/assign

Let's give this a quick try:

/close

@jpmcb
Copy link
Owner

jpmcb commented Jul 16, 2022

Interesting! Yeah, I think ignoring slash commands in code blocks makes sense. I'm happy to take this on 👏🏼

@jpmcb jpmcb reopened this Jul 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants