Skip to content

docs: add a pcapkit.__version__ entry and audit every :file: reference #902

Description

@JarryShaw

Describe the bug

Two documentation gaps, both from the maintainer on #719, verbatim:

we did not have a pcapkit.__version__ doc entry and those :file:xxx references need to double check if using the right paths and making sure the rendered links are clickable.

1. No pcapkit.__version__ API entry. Verified: every occurrence of __version__ in the docs is prose, not an API entry —

docs/source/pep.rst:890         "``pcapkit.__version__`` directly, derives ``PCAPKIT_PRERELEASE`` from"
docs/source/releasing.rst:14    "``pcapkit.__version__``. Everything else -- the ``v*`` tag, the GitHub"
docs/source/releasing.rst:22    "Bumping ``pcapkit.__version__`` is done by :file:`util/bump_version.py`"
docs/source/releasing.rst:28    "rewrites the ``__version__`` assignment in ``pcapkit/__init__.py``"
docs/source/releasing.rst:44    "Editing ``__version__`` by hand instead, without also moving"

pcapkit.__version__ is public API and is the single value the release pipeline reads, so it warrants a documented entry rather than only incidental mentions.

2. :file: references need auditing for correctness and clickability. Sweep every :file: role in docs/source/: does the path it names actually exist, is it correct relative to the repository root, and does it render as something a reader can click? The :file: role is semantic markup, not a link — so if the intent is a clickable reference to a file in the repository, :file: may be the wrong role entirely. Establish what the project wants and apply it consistently.

Expected behavior

A documented pcapkit.__version__ entry, and every :file: reference naming a real path with the intended rendering.

System information

Documentation only; no library version involved.

Additional context

Split out of #719, which is the prose sweep; these are structural. Related: #900 moves several of these pages into a subdirectory, which will change what a relative path in them resolves to — so if #900 lands first, re-run the :file: audit afterwards rather than before.

The Sphinx build currently emits 42 warnings, all pre-existing, so measure any delta rather than the total.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    docsPull requests that change documentation only (docs: subject prefix)fixPull requests that fix a defect (fix: subject prefix)

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions