-
Notifications
You must be signed in to change notification settings - Fork 28
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 reviewers automatically #4
Conversation
7c9b18e
to
b405c7e
Compare
Test in automation lab successful source PR: https://github.com/zeebe-io/automation-lab/pull/47 |
@pihme Please have a look |
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.
Looks great. Some minor points to consider before merging. None of those would be a bug, so feel free to implement or ignore them.
I think we should also start implementing tests going forward. |
@pihme I agree we should add tests. Should we do that before merging this or do you think it's out of scope? After applying the review hints I now still need to do an additional manual test to see that it still works. 😞 EDIT: I've also applied your hints. Please have a look 👍 |
@korthout Let's add the tests for the next change. |
This PR makes the action request reviewers for the newly created backport PRs. For this it uses the same reviewers as the PR that is backported.
closes #2