Check that pull requests follow the pull request template
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 58/100
- Tipo de issue
- Funcionalidade
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- github-actions, javascript
Direção de pesquisa
Start with scripts/sync-automation-template.js, automation-registry.yml, and the generated automation-group-pull-request-target.yml; compare the existing changelog.yml check and read docs/testing-automations.md. Test the changes in test-actions, including template-sync behavior and pull requests with missing, placeholder, valid, edited, and bot-authored bodies. Done means generated files are current and all acceptance criteria pass.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Overview
Contributors open pull requests that do not follow the pull request template. Reviewers then ask
for the missing information by hand.
Make the template reachable in every consumer repo, and check it through the automation entry point.
The check fails the run when the body does not follow the template.
Complexity: Medium
Target branch: main
Context
A sample of the most recent pull requests from non-members shows three separate failures:
| Repo | Sampled | No template sections | ## AI usage missing |
… placeholders left |
|---|---|---|---|---|
kolibri |
8 | 3 | 1 | 1 |
studio |
12 | 2 | 2 | 3 |
kolibri-design-system |
6 | 0 | 0 | 0 |
The template is not one check. A body can carry the headings and still leave ## AI usage empty,
which the template marks as required.
kolibri-design-system is the outlier, and the reason is where the file sits. It holds its own
template at .github/pull_request_template.md. morango and test-actions hold their own too.
kolibri, studio, le-utils, kolibri-app and kolibri-installer-android hold none, so they
fall back to the org default in this repo at .github/PULL_REQUEST_TEMPLATE.md.
GitHub applies an org default in the web form. A pull request opened with gh pr create, the REST
API, or a coding agent reads the working copy instead, and the org file is not in the working copy.
A contributor in a kolibri checkout therefore has no template to follow.
kolibri-design-system already enforces one section in .github/workflows/changelog.yml. It reads
the body, fails the job when the section is absent or still the placeholder text, and exempts
dependabot[bot]. It is the model for the check here.
The Change
Put the template in the checkout. Extend scripts/sync-automation-template.js so it carries
PULL_REQUEST_TEMPLATE.md to consumers alongside automation-template.yml. Skip any repo that
holds its own template. This prevents the failure rather than reporting it, because the file then
sits where every tool looks for it.
Add a pull-request-template automation. Add an entry to automation-registry.yml with a leaf
that reads github.event.pull_request.body. The leaf resolves the template the same way GitHub
does, repo copy first and org default second, then takes the required sections from it. The
generator places the job in automation-group-pull-request-target.yml.
The check reports three things:
- A required section heading is absent.
- A required section holds no text.
- A section still holds the
…placeholder.
Fail the run. The job exits non-zero, as changelog.yml does.
The job log is the only channel, so it must name every missing section rather than fail on the
first one.
Notes for the implementer
Trigger on pull_request_target types opened, edited and reopened, so a corrected body clears
the check. The template on: block currently carries labeled, opened and review_requested, so
edited and reopened are new. Every consumer needs the widened template, which the sync workflow
proposes on its next run.
Exempt bot authors, as changelog.yml does.
Out of Scope
- Changing the wording of the template.
- Posting a comment on the pull request. The failed check is the only report.
- Making the check required in branch protection. That is a repo setting, and it follows once the
check runs clean. - Changing the
kolibri-design-systemchangelog check.
Acceptance Criteria
- The sync workflow proposes
PULL_REQUEST_TEMPLATE.mdto consumers without their own copy, and
skipskolibri-design-system,morangoandtest-actions. -
automation-registry.ymlholds apull-request-templateentry, and the generated files are
up to date. - A body with no template sections fails the run.
- A body with the sections but an empty
## AI usagefails the run. - A body that still holds
…fails the run. - The log of a failed run names every missing section, not only the first.
- A body that follows the template passes.
- Editing a bad body to a good one turns the check green.
- Bot authors pass.
- The check reads the repo template in a repo that holds one, and the org default elsewhere.
Testing
Test in test-actions. See
testing the automations.
The contributor cases need an account outside the organization.
- Open a pull request with an empty body. Expect a failed check that names every missing section.
- Edit the body to follow the template. Expect the check to pass.
- Open a pull request that follows the template. Expect a passing check.
- Dispatch the sync workflow with
only: test-actionsanddry_run. Expect no template row,
becausetest-actionsholds its own copy.
References
- #97 added the sync workflow.
- #98 made
test-actionsthe place to test. - #101 grouped automation jobs by trigger event.
AI usage
I used Claude Code to sample recent contributor pull requests, to check which repos hold their own
template, and to read the kolibri-design-system changelog check. I decided the approach and
verified the findings against the repos.
- Linguagem predominante
- JavaScript
- Estrelas
- 1
- Forks
- 7
- Merge médio
- 1d 10h
- PRs com merge (30d)
- 4
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de learningequality/.github
-
Set up GitHub Discussions as the entry point for unplanned workTalvez já em andamento @akolson assumiu hoje. Aberta
learningequality/.github#105 · 1 responsável ·
-
github_actions
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 68/100
learningequality/.github#104 ·
-
dependencies github_actions
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 15/100
learningequality/.github#100 ·
-
Create process for unplanned work and automate PR handling based on linked issue assignmentTalvez livre de novo @akolson assumiu há 181 dias e não há nenhum pull request aberto. Abertagithub_actions
learningequality/.github#67 · 1 comentário · 1 responsável ·
Todas as issues de learningequality/.github
Issues semelhantes
-
Mend: dependency security vulnerability untriaged
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
opensearch-project/security-dashboards-plugin#2543 ·
Mantenedores costumam responder em até 1 dia
-
[quality] refresh-radar-reports.yml runs on ubuntu-latest while every other job pins ubuntu-24.04Abertaagent/quality hive/hosted-available-lke648397-260827-5n31 quality testing
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 90/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
godotengine/godot-website#1432 ·
-
Add: Mooz RetroAbertachannels:add check:passed
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
Mantenedores costumam responder em até 2 dias
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 90/100
Mantenedores costumam responder em até 1 dia