Skip to content

refactor(workflows): remove release build-ctk-ver input#1751

Merged
cpcloud merged 1 commit intoNVIDIA:mainfrom
cpcloud:remove-build-ctk-ver
Mar 12, 2026
Merged

refactor(workflows): remove release build-ctk-ver input#1751
cpcloud merged 1 commit intoNVIDIA:mainfrom
cpcloud:remove-build-ctk-ver

Conversation

@cpcloud
Copy link
Contributor

@cpcloud cpcloud commented Mar 11, 2026

Summary

  • derive the docs CTK build version from ci/versions.yml in the checked-out ref instead of requiring a manual build-ctk-ver input
  • remove redundant build-ctk-ver plumbing from the shared docs callers in release.yml, release-cuda-pathfinder.yml, and ci.yml
  • update the core release instructions to match the simplified release workflow inputs

Made with Cursor

@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Mar 11, 2026

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@cpcloud
Copy link
Contributor Author

cpcloud commented Mar 11, 2026

/ok to test

@github-actions

This comment has been minimized.

@cpcloud cpcloud requested a review from leofang March 11, 2026 14:24
@cpcloud cpcloud force-pushed the remove-build-ctk-ver branch from c7d2017 to 59831f3 Compare March 11, 2026 14:29
@cpcloud
Copy link
Contributor Author

cpcloud commented Mar 11, 2026

/ok to test

@cpcloud cpcloud requested a review from mdboom March 12, 2026 15:03
Remove the redundant release-time build-ctk-ver input and read the CTK build version from ci/versions.yml in the checked-out ref instead.

Made-with: Cursor
@cpcloud cpcloud force-pushed the remove-build-ctk-ver branch from 59831f3 to 6ef32dd Compare March 12, 2026 16:02
@cpcloud
Copy link
Contributor Author

cpcloud commented Mar 12, 2026

/ok to test

Copy link
Contributor

@mdboom mdboom left a comment

Choose a reason for hiding this comment

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

LGTM

@cpcloud cpcloud enabled auto-merge (squash) March 12, 2026 16:41
Copy link
Contributor

@Andy-Jost Andy-Jost left a comment

Choose a reason for hiding this comment

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

No issues

@cpcloud cpcloud merged commit 21f903b into NVIDIA:main Mar 12, 2026
86 checks passed
@github-actions
Copy link

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants