alunduil / alunduil/zfs-replicate
CI job/check names are intentional and stable enough to require as status checks
- Linguagem predominante
- Python
- Estrelas
- 24
- Forks
- 6
- Merge médio
- 3h 11min
- PRs com merge (30d)
- 49
Descrição
## Summary
Review the CI job names (and the check-run names they surface on a PR) and set
explicit names where the implicit job id is unclear or likely to churn, so a
subset can be safely pinned as required status checks on the default branch.
## Motivation
The Terraform baseline in `alunduil/alunduil-infrastructure` (alunduil/alunduil-infrastructure#25)
wants to gate this repo's default branch on required status checks via the
default-branch ruleset, but held off pending this review. Required checks are
matched by check-run *name*. Today those names are the implicit job ids
(`pre-commit`, `pytest`, `scripts`, `build`, `dependency-review`) because the
jobs don't set an explicit `name:`. Renaming or refactoring a job later would
change the check-run name and silently drop it from the required set, so the
names want a deliberate look before anything depends on them.
## Scope
- Inventory the jobs in `.github/workflows/ci.yml` and the exact check-run
names each surfaces on a PR.
- Decide which are stable, meaningful gates worth requiring (candidates:
`pytest`, `build`).
- Set an explicit `name:` on jobs where the implicit id is unclear or likely
to churn.
- Confirm each required-candidate name is unique on a PR (a name shared across
workflows can't be pinned to one).
## Acceptance criteria
- [ ] CI job/check names reviewed; explicit `name:` set where warranted
- [ ] The check names safe to require as branch-protection status checks are
written down (issue comment or the infra PR)
## Additional context
- Follow-up in infra: add `required_status_checks` to the `zfs-replicate`
module block once the names are settled.
Guia de contribuição
Direção de pesquisa
Start with .github/workflows/ci.yml and inventory each job and the check-run name it creates on a pull request. Review the candidates named in the issue, then confirm whether any required-candidate name is duplicated across workflows. Done means warranted jobs have explicit names and the safe-to-require names are recorded in an issue comment or the referenced infrastructure pull request.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- github-actions, python
- Domínio
- ci-cd, devops
- Tipo de issue
- Funcionalidade
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Status de atividade
- Ativa
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 74/100