adoroszlai opened a new pull request, #1195:
URL: https://github.com/apache/ratis/pull/1195
## What changes were proposed in this pull request?
Reduce duplication in CI workflow.
- Create a reusable workflow (`check.yml`) for a single, generic check.
- Refactor existing checks in `post-commit.yml` (except `coverage`, which
needs extra tokens) to call the reusable one. Minor changes as implementation
detail:
- Separate `repro` check from `compile`.
- Create separate `compile.sh` for `compile` check.
- Group simple checks (`author`, `checkstyle`, etc.) with matrix strategy as
`basic`.
- Split existing `post-commit.yml` into:
- `ci.yml`, has most of the logic, is triggered by workflow call
- `post-commit.yml`, triggered by `push`/`pull_request`, simply calls
`ci.yml`
This split is not strictly required, but I think the way GitHub UI
displays workflow runs is better this way (runs:
[before](https://github.com/adoroszlai/ratis/actions/runs/12341153183) /
[after](https://github.com/adoroszlai/ratis/actions/runs/12347469076) for
comparison).
https://issues.apache.org/jira/browse/RATIS-2210
## How was this patch tested?
CI:
https://github.com/adoroszlai/ratis/actions/runs/12452887900
--
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: issues-unsubscr...@ratis.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
