The GitHub Actions job "Required Checks" on texera.git/main has failed. Run started by GitHub user github-merge-queue[bot] (triggered by github-merge-queue[bot]).
Head commit for run: 9b740f9564dfb59f41509a4d82bc29cd2698753d / Yicong Huang <[email protected]> chore(ci): scope pyamber labeler globs to amber/ so bin/ changes only trigger infra (#5997) ### What changes were proposed in this PR? The `pyamber` label in `.github/labeler.yml` used repo-wide catch-all globs (`**/*.py`, `**/*requirements*.txt`). These also matched the tooling Python under `bin/` (`bin/local-dev-tui.py`, `bin/licensing/*.py`, `bin/local-dev/tests/*.py`), which now has its own lightweight `infra` job. As a result, a `bin/`-only PR was labeled `pyamber` on top of `infra` and wrongly pulled in the heavy `pyamber` + `amber-integration` stacks (`LABEL_STACKS.pyamber = ["amber-integration", "pyamber"]` in `required-checks.yml`). This PR scopes both globs to `amber/`: ```yaml pyamber: - changed-files: - any-glob-to-any-file: - 'amber/**/*.py' - 'amber/pyproject.toml' - 'amber/**/*requirements*.txt' ``` Coverage is preserved exactly for amber: - `amber/**/*.py` fully subsumes the old `amber/src/{main,test}/python/**` globs — those dirs contain only `.py` files (0 non-`.py` files) — and it still catches `amber/src/main/scala/.../aiassistant/*.py`, just like the old `**/*.py` did. - All three `requirements*.txt` files live under `amber/`, so `amber/**/*requirements*.txt` covers them. - The only other Python in the repo (`.github/scripts/*.py`) is already covered by the `ci` label, which triggers every stack. Net effect: `bin/` changes now trigger only the `infra` job, not `pyamber` / `amber-integration`. The stale `**/*.py` reference in the `engine` label comment is updated to `amber/**/*.py`. A repo-wide `!bin/**` negation was deliberately avoided: with labeler's `all-globs-to-all-files`, a PR touching both amber and bin Python would have the `pyamber` label wrongly suppressed. Scoping to `amber/` has no such edge case. ### Any related issues, documentation, discussions? Fixes #5999. Follow-up to #5978 (collapse dev label into infra and widen infra to `bin/**`), which added the `infra` job/label. ### How was this PR tested? This is a labeler-config-only change with no runtime code. Validated `.github/labeler.yml` parses as valid YAML, and manually traced the glob set against every tracked `*.py` / `*requirements*.txt` path in the repo to confirm amber coverage is unchanged and `bin/` no longer matches `pyamber`. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Opus 4.8 (1M context) Co-authored-by: Claude Opus 4.8 (1M context) <[email protected]> Report URL: https://github.com/apache/texera/actions/runs/28339274008 With regards, GitHub Actions via GitBox
