Skip to content

[doc-only] docs: drop cuda.coop, removed upstream by CCCL - #2430

Merged
mdboom merged 1 commit into
NVIDIA:mainfrom
aryanputta:docs/drop-removed-cuda-coop-links
Jul 27, 2026
Merged

[doc-only] docs: drop cuda.coop, removed upstream by CCCL#2430
mdboom merged 1 commit into
NVIDIA:mainfrom
aryanputta:docs/drop-removed-cuda-coop-links

Conversation

@aryanputta

@aryanputta aryanputta commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Closes #2429.

Summary

Deletes the cuda.coop entry from the top-level component list. CCCL removed
cuda.coop, so https://nvidia.github.io/cccl/unstable/python/coop.html returns
404 and cannot be repointed:

Why this matters beyond the stale entry

The entry renders into the sidebar of every generated page, so it fails the
lychee step of Docs / Build docs with 83 errors per run, on every open pull
request. main does not catch it because that step only runs on pull requests,
and pre-commit.ci does not catch it because lychee is in the ci: skip list.

Changes

Five deletions across three files, no additions:

  • README.md: the cuda.coop bullet
  • cuda_python/DESCRIPTION.rst: the cuda.coop bullet
  • cuda_python/docs/source/index.rst: the bullet, the link target, and the
    toctree entry

Validation

pre-commit run lychee on the three files, before and after:

# before
[404] https://nvidia.github.io/cccl/unstable/python/coop.html (at 13:15)   DESCRIPTION.rst
[404] https://nvidia.github.io/cccl/unstable/python/coop.html (at 24:16)   index.rst
[404] https://nvidia.github.io/cccl/unstable/python/coop.html (at 55:15)   index.rst
[404] https://nvidia.github.io/cccl/unstable/python/coop.html (at 8:3)     README.md
🔍 65 Total  🔗 20 Unique  ✅ 61 OK  🚫 4 Errors

# after
lychee...................................................................Passed

pre-commit run lychee --all-files also passes across all 198 authored .md
and .rst files, so no other dead link remains in the sources.

Also run and passing on the three files: check-spdx, end-of-file-fixer,
trailing-whitespace, mixed-line-ending, rst-backticks,
rst-directive-colons, rst-inline-touching-normal, check-case-conflict,
check-merge-conflict. git diff --check is clean.

docutils 0.23 on DESCRIPTION.rst reports the same single pre-existing
duplicate-target INFO before and after, and no new messages. On index.rst the
only change in docutils output is the removal of the deleted toctree line.

Notes for the reviewer

  • Titled [doc-only] so the test matrix is skipped. The doc job still runs and
    is still required by the aggregator, so the link fix is verified by CI.
  • CCCL now describes cuda.compute as device-level primitives, which is close to
    the role the deleted cuda.coop bullet described. I left our cuda.compute
    wording alone to keep this to the broken link. Happy to update it here if you
    would rather it be one change.
  • The sibling cuda.compute link also points under unstable/. It currently
    resolves, so I left it, but it carries the same risk of breaking on an upstream
    docs move.
  • Needs an assignee and a milestone from a maintainer for pr-metadata-check.

CCCL removed cuda.coop. Its docs page is gone, python/cuda_cccl/cuda/ now
contains only cccl and compute, and the CCCL Python index lists cuda.compute
alone. The link therefore returns 404 and cannot be repointed.

This fails the lychee link check in the Docs job on every open pull request,
because the entry renders into the sidebar of every generated page.

Signed-off-by: Aryan <aryansputta@gmail.com>
@copy-pr-bot

copy-pr-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@mdboom

mdboom commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

/ok to test df9ed66

@mdboom
mdboom enabled auto-merge (squash) July 27, 2026 19:47
@mdboom mdboom self-assigned this Jul 27, 2026
@mdboom mdboom added documentation Improvements or additions to documentation P0 High priority - Must do! labels Jul 27, 2026
@mdboom mdboom added this to the cuda.bindings next milestone Jul 27, 2026
@mdboom mdboom added the CI/CD CI/CD infrastructure label Jul 27, 2026
@github-actions

This comment has been minimized.

@mdboom
mdboom merged commit 94e3a76 into NVIDIA:main Jul 27, 2026
31 of 37 checks passed
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

Copy link
Copy Markdown
Doc Preview CI
Preview removed because the pull request was closed or merged.

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

Labels

CI/CD CI/CD infrastructure documentation Improvements or additions to documentation P0 High priority - Must do!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dead cuda.coop docs link fails the lychee check on every PR

2 participants