Skip to content

Commit c30c0f8

Browse files
committed
Infra: Document required CI dependencies
1 parent 5620dd2 commit c30c0f8

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

AGENTS.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,13 @@ lives under `pyiceberg/`, organized by concern rather than by engine:
6262
- Use existing test fixtures when possible.
6363
- We have a strong bias towards integration testing over mocks. Mocks should be avoided whenever possible and should only be used if similar, existing tests are using mocks.
6464

65+
### Required CI and Merge Queue
66+
67+
- Keep the `required_status_checks.contexts` list in `.asf.yaml` synchronized whenever a required job/check is added, renamed, or removed. Each entry is a job/check context name, not a workflow filename.
68+
- Verify that every required context reports for both `pull_request` and `merge_group`. Its producer workflow must run on both events.
69+
- When a workflow uses an aggregate required job, such as `python-ci-required`, keep `if: always()` and its `needs` list in sync with every job whose result should block merging.
70+
- Do not use `pull_request` path filters in workflows that produce required contexts. A skipped workflow does not report its required context, which blocks pull requests and causes Merge Queue entries to time out.
71+
6572
## Commands
6673

6774
- **Install / set up dev env:** `make install` (installs `uv`, syncs all extras, builds Cython, installs pre-commit hooks)

0 commit comments

Comments
 (0)