Opened 49 minutes ago
#37346 new Bug
PR quality checks do not recognize asterisk task list markers
| Reported by: | Prince Verma | Owned by: | |
|---|---|---|---|
| Component: | Core (Other) | Version: | dev |
| Severity: | Normal | Keywords: | PR, GitHub, checkboxes |
| Cc: | Prince Verma | Triage Stage: | Unreviewed |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
The PR quality checker does not recognize checked task list items that use an asterisk (*) instead of a hyphen (-), even though GitHub renders both formats as checked checkboxes.
I encountered this while updating the description of my documentation PR:
https://github.com/django/django/pull/21935
The description used "* [x]" for checked items. GitHub displayed them as checked, but the quality checker reported missing AI disclosure and an incomplete checklist. Replacing the asterisk list markers with hyphens resolved these errors.
Steps to reproduce:
- Start with a PR description that passes the AI disclosure and checklist checks, including exactly one selected AI disclosure option, the required AI usage explanation where applicable, and the first five checklist items checked.
- Replace the leading hyphen list markers in the AI disclosure and Checklist sections with asterisks, preserving the checkbox states and all other content.
- Run the AI disclosure and checklist checks against the modified description.
Expected behavior:
The checks should recognize asterisk task list markers and produce the same results as for equivalent hyphen task list markers.
Actual behavior:
The checks report missing AI disclosure and an incomplete checklist.
The checkbox matching in check_ai_disclosure() and check_checklist() in scripts/pr_quality/check_pr.py currently expects hyphen markers. The filtering of checkbox lines from the AI usage explanation also assumes hyphen markers.
Forum discussion:
https://forum.djangoproject.com/t/pr-quality-checks-reject-checked-items-using-asterisk-list-markers/45990
In that discussion, CodenameTim recommended opening a Trac ticket.
I would be interested in working on a fix with regression tests.