ci/github-script/reviewers: add TODO about future optimization
We still use a few too many API requests by checking team members for collaborator status - we can improve on that in the future.
This commit is contained in:
@@ -75,6 +75,9 @@ async function handleReviewers({
|
||||
const reviewers = (
|
||||
await Promise.all(
|
||||
Array.from(new_reviewers, async (username) => {
|
||||
// TODO: Restructure this file to only do the collaborator check for those users
|
||||
// who were not already part of a team. Being a member of a team makes them
|
||||
// collaborators by definition.
|
||||
try {
|
||||
await github.rest.repos.checkCollaborator({
|
||||
...context.repo,
|
||||
|
||||
Reference in New Issue
Block a user