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

added team_reviewers #493

Merged
merged 5 commits into from
Dec 23, 2021
Merged

added team_reviewers #493

merged 5 commits into from
Dec 23, 2021

Conversation

adzienis
Copy link
Contributor

Similar to #482, but we realized we would also like to add teams as reviewers as well.

.github/workflows/test.yml Outdated Show resolved Hide resolved
index.js Outdated Show resolved Hide resolved
@adzienis
Copy link
Contributor Author

Combined reviewers and team_reviewers into one branch

Copy link
Owner

@gr2m gr2m left a comment

Choose a reason for hiding this comment

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

Can you update the action.yml?

index.js Outdated Show resolved Hide resolved
Copy link
Owner

@gr2m gr2m left a comment

Choose a reason for hiding this comment

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

can you not force-push your commits? The PR can be messy, I'll squash & merge anyway. Just add more commits, it will make it easier to review the changes from my last review

index.js Outdated Show resolved Hide resolved
.github/workflows/test.yml Outdated Show resolved Hide resolved
@adzienis
Copy link
Contributor Author

Removed team reviewer check in test.yml

Copy link
Owner

@gr2m gr2m left a comment

Choose a reason for hiding this comment

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

Thanks :) If you could give it a try once the new version is released, that'd be great

@gr2m gr2m merged commit 466b1b8 into gr2m:master Dec 23, 2021
@github-actions
Copy link
Contributor

🎉 This PR is included in version 1.9.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@adzienis
Copy link
Contributor Author

Sure, thanks for the great feedback, and very quick responsiveness as well. apologies, but I'll be able to test it in a couple hours from now; i'll comment here with the status

@gr2m
Copy link
Owner

gr2m commented Dec 23, 2021

no rush :)

@adzienis
Copy link
Contributor Author

well, the reviewers does seem to work. team_reviewers is being a bit tough rn. GH isn't showing any sort of teams I added to a private repo/org (as a scratch/test), even with two dummy users added to a team, so it's tougher to test that. No errors/exceptions thrown though when the team is listed in the workflow, just the team isn't assigned, but as of now, I can't manually assign/get them to show up anyway (even after fiddling with the normal settings to). possibly need time to sync? our team will most likely run this next action on Jan 2nd

@gr2m
Copy link
Owner

gr2m commented Dec 23, 2021

It's possible that the team_reviewers setting will not work if you set GITHUB_TOKEN to ${{ secrets.GITHUB_ACTION }}, because you might need the members:read permission for organization, which the token provided from GitHub Actions does not have. Try setting it to a personal access token with the repo and read:org scopes.

But you say that you yourself cannot assign a pull request reviewer to a team? The team needs read access to the repository

@adzienis
Copy link
Contributor Author

Yeah, the team actually has admin privileges; which still doesn’t work strangely. I will try what you suggested with the token.

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

Successfully merging this pull request may close these issues.

2 participants