yuqi1129 opened a new pull request, #14190:
URL: https://github.com/apache/gravitino/pull/14190
### What changes were proposed in this pull request?
- `.asf.yaml`: drop `Fork CI` and `ASF Allowlist Check` from
`required_status_checks.contexts`; keep `strict: true` and the review
requirement (the same protection as before #13049).
- CONTRIBUTING: describe both checks as not enforced, and ask committers to
merge only after they pass, or to comment on the PR with what they verified
when they must merge earlier.
### Why are the changes needed?
A PR that changes CI definitions needs a trusted Fork CI run on the shared
upstream runners to turn **Fork CI** green. Those runners queued for hours, so
urgent CI fixes could not be merged.
Fix: #14189
### Does this PR introduce _any_ user-facing change?
Committers: the two checks no longer block the merge button; they remain
visible on every PR.
### How was this patch tested?
`.asf.yaml` parsed with a YAML parser: `protected_branches.main` =
`{required_status_checks: {strict: true}, required_pull_request_reviews:
{dismiss_stale_reviews: true, required_approving_review_count: 1}}`. ASF Infra
applies the settings when the change lands on `main`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
