Context
We have shame/ labels (shame/description, shame/assignee, shame/reviewer, shame/labels) for PRs that are missing required metadata. Currently pr-hygiene just fails the check -- it doesn't apply the shame labels.
Proposal
Extend cli/pr-hygiene (or add a post-check step) to automatically apply the appropriate shame/* labels to PRs that fail hygiene checks:
- Missing description ->
shame/description
- No assignee ->
shame/assignee
- No reviewer ->
shame/reviewer
- No labels ->
shame/labels
Labels should be added via the Forgejo API. When the issue is fixed and the check re-runs, the corresponding shame label should be removed.
Notes
- Requires a token with label-write permissions (already available as
BLIZZ_PR_TOKEN)
- Could also post a comment with a disapproving message for extra motivation
- Consider whether the check should still hard-fail or just warn + shame
Context
We have
shame/labels (shame/description,shame/assignee,shame/reviewer,shame/labels) for PRs that are missing required metadata. Currentlypr-hygienejust fails the check -- it doesn't apply the shame labels.Proposal
Extend
cli/pr-hygiene(or add a post-check step) to automatically apply the appropriateshame/*labels to PRs that fail hygiene checks:shame/descriptionshame/assigneeshame/reviewershame/labelsLabels should be added via the Forgejo API. When the issue is fixed and the check re-runs, the corresponding shame label should be removed.
Notes
BLIZZ_PR_TOKEN)