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

Idempotency #36

Open
shelbyd opened this issue Mar 29, 2023 · 1 comment
Open

Idempotency #36

shelbyd opened this issue Mar 29, 2023 · 1 comment

Comments

@shelbyd
Copy link

shelbyd commented Mar 29, 2023

Currently, each time this is run it will potentially add new reviewers. I'd like to run this after all our tests pass.

It would be great if this could check the current set of reviewers on the PR and only assign to fulfill the constraints. That way, running it subsequently would result in no new assignments.

This seems like a pretty substantial change, so would probably be better to have someone more familiar with the project do it.

@uesteibar
Copy link
Owner

I agree it'd be useful yeah. Tho I doubt I'll have time to do it myself in the near future :(

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

No branches or pull requests

2 participants