Yicong-Huang opened a new issue, #5964:
URL: https://github.com/apache/texera/issues/5964
### Task Summary
`Check License Headers` lives in its own one-step workflow
(`.github/workflows/check-header.yml`, `Release Auditing`) that triggers on
every PR. It overlaps in purpose with the lightweight `infra` job in
`build.yml` — both run project-wide lint-style checks, both should fire on
every PR, both are cheap.
Fold the license-header check into the `infra` job as an additional step,
and delete the standalone workflow. Same skywalking-eyes action, same
`.licenserc.yaml`, just one fewer top-level check in PR status:
```
Before: After:
─────────────────────────────────────
─────────────────────────────────────
Release Auditing build / infra (ubuntu, macos)
└─ Check License Headers ├─ Check License Headers
├─ Run shell smoke tests
build / infra (ubuntu, macos) └─ Run Python unit tests
├─ Run shell smoke tests
└─ Run Python unit tests
```
Also: the `infra` job is currently gated by `inputs.run_infra` (only runs
when `bin/local-dev*` files change). Drop that gate so it runs on every PR —
license headers can be missing anywhere, not just in `bin/local-dev*`.
### Task Type
- [x] DevOps / Deployment / CI
--
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]
