Automattic / Automattic/jetpack
Repo Gardening > Check description: adjust wording and checks
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
The task currently runs a number of checks and outputs a message in a comment on each PR:
https://github.com/Automattic/jetpack/blob/b76a27c4f3700a310aad288833eeb7a6c3a496fc/projects/github-actions/repo-gardening/src/tasks/check-description/readme.md
There are 4 things we need to improve with this task:
```[tasklist]
### Tasks
- [ ] Make the wording of the comment clearer, that those things are **required** before you can proceed with merging the PR.
- [ ] Update the part about "Needs Review" and "Needs Team Review" ; it's outdated and still references Crew.
- [ ] We should take that opportunity to re-work the Status labels requirements. We may not always need a status label if we get rid of the "Needs team review" label for example.
- [ ] Improve the checks we run for testing instructions / privacy section: we need to do more than just check for the heading.
```
*******
Related references:
- p1740161892407389/1740155311.575309-slack-CDLH4C1UZ
- p1740158150216529/1740154130.612439-slack-CDLH4C1UZ
- p9dueE-8nS-p2#comment-10574
- pf4qpu-u1-p2
Contributor guide
Assessment
This issue has not been assessed yet.