Skip to content
This repository has been archived by the owner on Aug 27, 2024. It is now read-only.

Approval/veto regexes only work for start of comment #582

Open
itsthejb opened this issue Jul 17, 2019 · 1 comment
Open

Approval/veto regexes only work for start of comment #582

itsthejb opened this issue Jul 17, 2019 · 1 comment

Comments

@itsthejb
Copy link

itsthejb commented Jul 17, 2019

Hi there,

Trying to use the following config:

approvals:
  minimum: 2
  pattern: ":\\+1:|:pray:|:shipit:|πŸ‘|πŸ™|βœ…"
  veto:
    pattern: ":\\-1:|πŸ‘Ž|❌" # must contain thumbs down or other

i.e. the approvals/vetos can occur anywhere in the comment, not necessarily the start (by removing anchors). This however, appears not to work, even though there would seem to be no limitation on this here.

It would be nice to have this for more expressive messages.

I did a quick test, and it seems (by the unit tests) that it should work: https://github.com/zalando/zappr/compare/master...itsthejb:test-approval-patterns?expand=1. However, in practice it seems not

Hope someone can help!

@itsthejb
Copy link
Author

Edit: It seems the issue is that Zappr isn't reading the updated config (on master)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant