ohemorange
6ee19bac55
Allow notification of two reviewers being assigned to a PR and two issue assignees ( #10345 )
...
Fixes https://github.com/certbot/certbot/issues/10344
You can see this working in the mattermost "Test" channel, where I ran
this code from my test repo.
The documentation for the PR reviewer syntax is here:
https://docs.github.com/en/webhooks/webhook-events-and-payloads?actionType=review_requested#pull_request
We now no longer notify on PR assignees. But I think that is the correct
behavior.
This PR also fixes a bug in the issue assigned notification code, and
now lets you see when two different people were assigned. That
documentation is here:
https://docs.github.com/en/webhooks/webhook-events-and-payloads#
After this is in, I'll make the same changes to the josepy repo.
You can see the `if` syntax here:
https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/events-that-trigger-workflows
```
on:
pull_request:
types: [review_requested]
jobs:
specific_review_requested:
runs-on: ubuntu-latest
if: ${{ github.event.requested_team.name == 'octo-team'}}
steps:
- run: echo 'A review from octo-team was requested'
```
---------
Co-authored-by: Brad Warren <bmw@users.noreply.github.com>
2025-06-23 17:08:21 +00:00
