Automattic / Automattic/jetpack

Repo Gardening > Check description: adjust wording and checks

Open
#41,959 1 comment 0 reactions 1 assignee Claimed by @jeherve View on GitHub
[Action] Repo Gardening [Pri] Normal Actions Enhancement
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.