Skip to content

docs: cite each number by its kind, and fix heading and Mermaid forms - #2626

Merged
JarryShaw merged 1 commit into
mainfrom
docs/2561-citation-forms
Nov 2, 2026
Merged

JarryShaw merged 1 commit into
mainfrom
docs/2561-citation-forms

Conversation

@JarryShaw

Copy link
Copy Markdown
Owner

What is the purpose of your pull request?

  • docs — documentation only changes

Description of your pull request and other information

Each :issue:/:pr: role names a number of the right kind, and each ruling made in a PR's review is cited by the issue that PR closes, per #2599. Four headings are renamed or re-cased to match documentation.rst, and the Mermaid labels escape their leading underscores.

The test from #1103 checks every role's kind against tests/project/issue_pr_roles_fixtures/kinds.json. Numbers above the snapshot's maximum are skipped, which is the option the owner chose on #2561. util/issue_pr_kinds.py regenerates the snapshot. The weekly Issue and PR Kinds workflow runs it and opens or updates a bot PR when the snapshot changes. It copies Python Versions' Release App pattern behind the github-release environment, and it never pushes main.

A Sonnet cross-review returned GOOD TO GO, after re-checking each citation against GitHub and the workflow line by line. The Sphinx warning set is identical to main's.

Closes #2561

…Mermaid forms (#2561)

- Cite a ruling from a pull request's review by the issue it closes, or by
  :pr: with no issue holding it, on the convention, pep, workflows and
  releasing pages and in the vendor/default and ip_pseudo doc text.
- Rename finite-verb headings on the warnings, logging, ext and layer
  placement pages, repoint the protocols graph's click target, re-case
  lowercase content words in const, vendor and protocol titles, and
  escape the leading underscores in two Mermaid node labels.
- The 1103 test checks each role's kind against a snapshot, skipping
  numbers above it; util/issue_pr_kinds.py regenerates it, and the new
  Issue and PR Kinds workflow proposes a refresh as a pull request.

Closes #2561.
@JarryShaw JarryShaw added this to the PyPCAPKit v1.6 milestone Nov 2, 2026
@JarryShaw JarryShaw added docs Pull requests that change documentation only (docs: subject prefix) ci Pull requests that change CI or workflow configuration (ci: subject prefix) review: good-to-go Cross-review at the current head says ready; CI state is separate labels Nov 2, 2026
@JarryShaw JarryShaw added ci Pull requests that change CI or workflow configuration (ci: subject prefix) review: good-to-go Cross-review at the current head says ready; CI state is separate labels Nov 2, 2026
@JarryShaw

Copy link
Copy Markdown
Owner Author

GOOD TO GO at 1278c1172, from a Sonnet cross-review:

  • Every changed citation was re-checked against GitHub for its kind and for what each PR closes.
  • The kinds test behaves as option 2: wrong kind fails, a Discussion number fails, and numbers above the snapshot are skipped.
  • The workflow was read line by line. Permissions are {} at the top, with read-only detect and App-token pr behind github-release. There is no injection path, and it never pushes main.
  • The Sphinx warning set is identical to main's.
  • A cost to weigh: almost every Monday will propose a refresh PR, because new numbers keep arriving, and each one needs a github-release approval.

@JarryShaw JarryShaw moved this to In review in PyPCAPKit Nov 2, 2026
@JarryShaw
JarryShaw merged commit 5def5cf into main Nov 2, 2026
61 of 62 checks passed
@JarryShaw
JarryShaw deleted the docs/2561-citation-forms branch November 2, 2026 04:32
@github-project-automation github-project-automation Bot moved this from In review to Done in PyPCAPKit Nov 2, 2026
@JarryShaw JarryShaw removed the review: good-to-go Cross-review at the current head says ready; CI state is separate label Nov 2, 2026
JarryShaw added a commit that referenced this pull request Nov 2, 2026
… kinds into 1.6.0 (#2449)

- Added: pad_length and PaddingField's align/after form (#2624), and
  pcapkit.utilities.contextvars (#2623); the old private names get no
  entry, as for the other private API.
- Changed: the weekly Issue and PR Kinds workflow, beside the milestone
  workflow's entry (#2626).
- The conventions entry records where a context variable lives (#2623),
  and the role-kind citation rule with its test (#2626).
- No entry for #2626's heading and Mermaid fixes, #2564 or fa9af52.
- CHANGELOG.md regenerated; util/changelog_md.py --check passes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Pull requests that change CI or workflow configuration (ci: subject prefix) docs Pull requests that change documentation only (docs: subject prefix)

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

docs: citation, heading and Mermaid forms that depart from documentation.rst

1 participant