Skip to content

docs: update native-image CI guidance for Linux-only per-PR build#1000

Open
nielspardon wants to merge 1 commit into
substrait-io:mainfrom
nielspardon:docs-update-native-image-ci-note
Open

docs: update native-image CI guidance for Linux-only per-PR build#1000
nielspardon wants to merge 1 commit into
substrait-io:mainfrom
nielspardon:docs-update-native-image-ci-note

Conversation

@nielspardon

Copy link
Copy Markdown
Member

Summary

Updates the AGENTS.md guidance on monitoring PR checks to match the CI change made in #997.

The old text described the macOS Build Isthmus Native Image job as the per-PR "long pole" that needs: the java + integration jobs. As of #997 that's no longer true:

  • The per-PR native-image job (pr.yml) builds Linux only, uses --quick-build-native, and runs in parallel with the java/integration jobs (the needs: was removed) — so it's fast and not a long pole.
  • The macOS native image is no longer built on PRs. It runs out-of-band in .github/workflows/native-image-macos.yml (push to main, a weekly backstop, and workflow_dispatch), fully optimized to mirror release.yml.

This is a docs-only change (guidance for agents/contributors); no code or workflow behavior is affected.

🤖 Generated with AI

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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant