Skip to content

docs: record how to transplant cutile-rs pull requests - #1393

Merged
elibol merged 1 commit into
NVIDIA:mainfrom
roivanov:docs/cutile-pr-transplant
Oct 7, 2026
Merged

elibol merged 1 commit into
NVIDIA:mainfrom
roivanov:docs/cutile-pr-transplant

Conversation

@roivanov

@roivanov roivanov commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

AGENTS.md now records the cutile-rs transplant procedure used for NVlabs/cutile-rs#305 / NVIDIA/cuda-rust#1391, so the next replay can follow the same steps.

Changes

  • Keep the original author as author, committer, and the only sign-off.
  • Prefix paths with cutile-rs/, then cherry-pick so the host-crate move to the repository root is applied again.
  • Leave code conflicts for a developer. A changelog note that belongs under Unreleased may be resolved.
  • Push that feature branch to NVIDIA/cuda-rust and cross-link the source pull request. This is the one exception to the fork-only push rule.

Testing

Documentation only. No code or tests to run.

  • just -f cuda-oxide/Justfile check passes (the local mirror of CI: fmt, clippy, tests, guards, docs)
  • cargo oxide run <example> passes, or cuda-oxide/scripts/smoketest.sh -o '^<example>$'
  • New example added (if applicable)

Checklist

  • All commits signed off (git commit -s)
  • SPDX headers on new source files

Questions about the review? Ping us in #contributors on Discord.

Keep the original author, follow the host-crate move, and cross-link
the source pull request so the next replay does not depend on chat history.

Signed-off-by: Roman Ivanov <rivanov@nvidia.com>

@elibol elibol left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@elibol
elibol merged commit 9120bb9 into NVIDIA:main Oct 7, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants