shen/brew
1
0
Fork 0
mirror of https://github.com/Homebrew/brew.git synced 2024-11-25 16:33:34 +08:00
Code Issues Actions 11 Packages Projects Releases Wiki Activity

Merge pull request #18778 from elijaholmos/too-many-prs-saml-patch

Browse Source 
fix: `too_many_open_prs()` fails without SSO access
This commit is contained in:
Mike McQuaid 2024-11-18 08:49:53 +00:00 committed by GitHub
commit a12a215239
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 17 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
Library/Homebrew/utils/github.rb
View File

@ -924,22 +924,27 @@ module GitHub
homebrew_prs_count = 0
API.paginate_graphql(query) do |result|
data = result.fetch("viewer")
github_user = data.fetch("login")
begin
API.paginate_graphql(query) do |result|
data = result.fetch("viewer")
github_user = data.fetch("login")
# BrewTestBot can open as many PRs as it wants.
return false if github_user.casecmp?("brewtestbot")
# BrewTestBot can open as many PRs as it wants.
return false if github_user.casecmp?("brewtestbot")
pull_requests = data.fetch("pullRequests")
return false if pull_requests.fetch("totalCount") < MAXIMUM_OPEN_PRS
pull_requests = data.fetch("pullRequests")
return false if pull_requests.fetch("totalCount") < MAXIMUM_OPEN_PRS
homebrew_prs_count += pull_requests.fetch("nodes").count do |node|
node.dig("baseRepository", "owner", "login").casecmp?("homebrew")
homebrew_prs_count += pull_requests.fetch("nodes").count do |node|
node.dig("baseRepository", "owner", "login").casecmp?("homebrew")
end
return true if homebrew_prs_count >= MAXIMUM_OPEN_PRS
pull_requests.fetch("pageInfo")
end
return true if homebrew_prs_count >= MAXIMUM_OPEN_PRS
pull_requests.fetch("pageInfo")
rescue => e
# Ignore SAML access errors (https://github.com/Homebrew/brew/issues/18610)
raise unless e.message.include?("Resource protected by organization SAML enforcement")
end
false