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

Revert "Test BC breaks of all banners on review request" #668

Merged
merged 1 commit into from
Dec 17, 2024

Conversation

gbirke
Copy link
Member

@gbirke gbirke commented Dec 17, 2024

The change in #667 runs the expensive action for every reviewer selected. This eats too much in our GitHub CI budget. The ideal behavior should be to run only once. We can achieve this by querying the GitHub API (with the github-script container for the list of reviewers and only executing when the user name in the current run of the action matches the first user name from the list of reviewers in the API. Until we have time to implement this, we should revert the expensive change

Reverts #667

@moiikana moiikana merged commit e542d0f into main Dec 17, 2024
1 check passed
@moiikana moiikana deleted the revert-667-test-all-banners-on-review branch December 17, 2024 10:34
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

Successfully merging this pull request may close these issues.

2 participants