Skip to content

Conversation

@gevtushenko
Copy link
Contributor

No description provided.

@copy-pr-bot
Copy link

copy-pr-bot bot commented Jan 12, 2026

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.

@github-actions
Copy link
Contributor

👋 Thank you for your contribution!

This pull request is from a forked repository so GitHub Actions will not be able to run CI. A maintainer will review your changes shortly and manually trigger the CI.

@maintainers Please review this PR when you have a chance and follow the instructions in the CONTRIBUTING.md file to trigger the CI.

@github-actions
Copy link
Contributor

⚠️ Unsigned Commits Detected

This pull request contains 1 unsigned commit(s):

  • 657cefe CUDA C++ Contriguting Guide

All commits must be signed before this PR can be merged. Please sign your commits by following these steps:

Option 1: SSH Key Signing (Recommended)

  1. Add your SSH key to GitHub (if not already done):

  2. Configure Git to use SSH signing:

    git config --global gpg.format ssh
    git config --global user.signingkey ~/.ssh/id_ed25519.pub  # or your key path
    git config --global commit.gpgsign true
  3. Re-sign your commits:

    git rebase -i HEAD~1 --exec "git commit --amend --no-edit -S"
    git push --force

Option 2: GPG Key Signing

  1. Generate a GPG key: GitHub Docs

  2. Add the GPG key to GitHub: GitHub Docs

  3. Configure Git to sign commits:

    git config --global user.signingkey YOUR_GPG_KEY_ID
    git config --global commit.gpgsign true
  4. Re-sign your commits (same as above):

    git rebase -i HEAD~1 --exec "git commit --amend --no-edit -S"
    git push --force

For more details, see GitHub's commit signature verification docs.


This comment will be updated when you push new commits.

@gevtushenko gevtushenko force-pushed the cpp-contributing-guide branch from 657cefe to a6af2d0 Compare January 12, 2026 04:22
Copy link

@smish-nvidia smish-nvidia left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@brycelelbach brycelelbach changed the title CUDA C++ Contriguting Guide CUDA C++ Contributing Guide Jan 23, 2026
Copy link
Collaborator

@brycelelbach brycelelbach left a comment

Choose a reason for hiding this comment

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

Looks good, I left a few minor notes.

@brycelelbach
Copy link
Collaborator

@gevtushenko the link check is failing because it's failing to find the theoretical source file you reference in the doc.

Error: R] <file:///home/runner/work/accelerated-computing-hub/accelerated-computing-hub/tutorials/cuda-cpp/Solutions/no-magic-execution-space-changes.cpp> | Cannot find file: File not found. Check if file exists and path is correct

You should add that file path to $REPO_ROOT/brev/.lycheeignore.

@brycelelbach brycelelbach self-requested a review January 23, 2026 15:12
Copy link
Collaborator

@brycelelbach brycelelbach left a comment

Choose a reason for hiding this comment

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

Please fix the false positive link check failure. It's failing to find the theoretical source file you reference in the doc.

Error: R] <file:///home/runner/work/accelerated-computing-hub/accelerated-computing-hub/tutorials/cuda-cpp/Solutions/no-magic-execution-space-changes.cpp> | Cannot find file: File not found. Check if file exists and path is correct

You should add that file path to $REPO_ROOT/brev/.lycheeignore.

@brycelelbach brycelelbach force-pushed the cpp-contributing-guide branch from 951d56d to 22d4257 Compare February 4, 2026 18:50
@brycelelbach
Copy link
Collaborator

/ok to test 22d4257

Copy link
Collaborator

@brycelelbach brycelelbach left a comment

Choose a reason for hiding this comment

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

Looks good to go.

@brycelelbach brycelelbach enabled auto-merge (rebase) February 4, 2026 18:52
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.

4 participants