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

ci: Automate requesting review #656

Closed
wants to merge 1 commit into from

Conversation

omarluq
Copy link
Contributor

@omarluq omarluq commented Aug 1, 2024

adds a simple github action that automates requesting review from @jorgemanrubia and @seanpdoyle on PR's that passes CI

adds a simple github action that automates requesting review from @jorgemanrubia and @seanpdoyle on PR's that passes CI
@omarluq omarluq marked this pull request as draft August 1, 2024 16:15
with:
github-token: ${{secrets.GITHUB_TOKEN}}
script: |
const reviewers = ['seanpdoyle', 'jorgemanrubia'];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm flattered to be included in this list.

With that being said, I am not a project maintainer, and do not have permissions to merge or close pull requests.

Copy link
Contributor Author

@omarluq omarluq Aug 1, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your opinions and feedback are valuable imho and defiantly should be on the list.
However hardcoding usernames on github actions is bad practice, this pr is to stir up a convo about this but ideally there should be a github team that the maintainers controller membership to and gets tagged instead IMHO

@omarluq omarluq closed this Sep 16, 2024
@omarluq omarluq deleted the automate-reviews branch September 16, 2024 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants