#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:

  1. 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.
  2. Replace the leading hyphen list markers in the AI disclosure and Checklist sections with asterisks, preserving the checkbox states and all other content.
  3. 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.

Change History (0)

Note: See TracTickets for help on using tickets.
Back to Top