Skip to content

Fetch pull request labels without authentication - #4225

Merged
amomchilov merged 3 commits into
mainfrom
codex/check-labels-rest-api
Oct 26, 2026
Merged

amomchilov merged 3 commits into
mainfrom
codex/check-labels-rest-api

Conversation

@soutaro

@soutaro soutaro commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

The required check-labels job can fail on correctly labeled PRs when its authenticated API request is rejected by the organization's IP allow list. Fetch public PR data without authentication and stop on HTTP errors before label validation to avoid a secondary JSON parsing error. Label validation errors also list documentation, chore, and dependencies for changes that do not require release notes.

Fixes #4219.

@soutaro
soutaro requested a review from a team as a code owner October 7, 2026 04:26
@soutaro soutaro added chore Chore task and removed chore Chore task labels Oct 7, 2026
Comment thread .github/workflows/require_labels.yml Outdated
@soutaro
soutaro requested a review from amomchilov October 9, 2026 02:25
@vinistock

Copy link
Copy Markdown
Member

All GH actions are unfortunately subject to failing because of the IP restriction thing. It's a flaky issue and inconvenient, but it would require us to patch every single action across all of our repositories, so I don't think it's really worth it.

@amomchilov
amomchilov merged commit fb2e794 into main Oct 26, 2026
40 checks passed
@amomchilov
amomchilov deleted the codex/check-labels-rest-api branch October 26, 2026 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Chore task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

check-labels intermittently fails due to the organization IP allow list

3 participants