Skip to content

Commit e37388c

Browse files
committed
docs: update native-image CI guidance for Linux-only per-PR build
The per-PR native-image job no longer builds the macOS image and no longer depends on the java/integration jobs, so the "macOS native image is the PR long pole" guidance is stale. Reflect the current setup: PRs build the Linux native image only (with --quick-build-native, in parallel with the other jobs), while the macOS native image is built out-of-band in native-image-macos.yml on push to main, a weekly schedule, and workflow_dispatch.
1 parent a107b72 commit e37388c

1 file changed

Lines changed: 8 additions & 5 deletions

File tree

AGENTS.md

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -205,8 +205,11 @@ compile — they have their own visitor implementors:
205205
proto message usually needs: POJO + visitor wiring + both proto converters + a
206206
round-trip test, and often `ExpressionCreator` factories and `dsl/SubstraitBuilder`
207207
helpers for ergonomics.
208-
- When monitoring PR checks, budget for a long tail: the **macOS `Build Isthmus Native
209-
Image`** job is the long pole — it `needs:` the `java` + `integration` jobs (so it
210-
starts late), then AOT-compiles for ~15–20 min on a slower macOS runner. A PR staying
211-
yellow after the other checks pass usually just means that job is still running, not
212-
that it's stuck or failing.
208+
- When monitoring PR checks: the per-PR native-image job (`Build Isthmus Native Image` in
209+
`pr.yml`) builds **Linux only**, uses `--quick-build-native`, and runs **in parallel**
210+
with the `java` + `integration` jobs (no `needs:`), so it's fast and not a long pole. The
211+
**macOS** native image is *not* built on PRs — it runs out-of-band in
212+
`.github/workflows/native-image-macos.yml` (push to `main`, a weekly backstop, and
213+
`workflow_dispatch`), fully optimized to mirror `release.yml`. So a PR that stays yellow
214+
isn't waiting on a macOS native build, and macOS-specific native regressions surface on
215+
`main` or the weekly run rather than on the PR.

0 commit comments

Comments
 (0)