-
Notifications
You must be signed in to change notification settings - Fork 112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add section about SSH signing keys to developer docs. #1755
Conversation
e6d069a
to
4126128
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requesting changes for now as I'm on the fence about putting this in the contributing guide. I don't want to overwhelm new contributors with a bunch of stuff that it seems like they need to do. Especially with this one where it strictly isn't really necessary.
If we do add it, I'd prefer to put in the CI details section here: https://github.com/NVIDIA/cccl/blob/main/ci-overview.md#ci-workflow-details
CONTRIBUTING.md
Outdated
```bash | ||
git commit -m "Brief description of the change" | ||
``` | ||
|
||
### SSH Signing Keys | ||
|
||
CCCL projects require commits to be signed with a cryptographic key. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comment: This isn't quite true. Signed commits are only necessary for internal contributors who want CI to run automatically and not require /ok to test
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, ok, I'd misunderstood that nuance. I'll gladly move it to the ci docs, as long as I have something concise to link (and refer back to myself 😄)
🟩 CI Results [ Failed: 0 | Passed: 274 | Total: 274 ]
|
# | Runner |
---|---|
232 | linux-amd64-cpu16 |
24 | linux-arm64-cpu16 |
18 | windows-amd64-cpu16 |
👃 Inspect Changes
Modifications in project?
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
[skip-tests] Co-authored-by: Michael Schellenberger Costa <[email protected]>
602d34a
to
3ad8979
Compare
🟩 CI Results [ Failed: 0 | Passed: 274 | Total: 274 ]
|
# | Runner |
---|---|
232 | linux-amd64-cpu16 |
24 | linux-arm64-cpu16 |
18 | windows-amd64-cpu16 |
👃 Inspect Changes
Modifications in project?
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
[skip-tests]
806e437
to
575db8e
Compare
🟩 CI Results [ Failed: 0 | Passed: 274 | Total: 274 ]
|
# | Runner |
---|---|
232 | linux-amd64-cpu16 |
24 | linux-arm64-cpu16 |
18 | windows-amd64-cpu16 |
👃 Inspect Changes
Modifications in project?
Project | |
---|---|
+/- | CCCL Infrastructure |
libcu++ | |
CUB | |
Thrust | |
CUDA Experimental |
Modifications in project or dependencies?
Project | |
---|---|
+/- | CCCL Infrastructure |
+/- | libcu++ |
+/- | CUB |
+/- | Thrust |
+/- | CUDA Experimental |
No description provided.