ci: add request-nvskills-ci.yml workflow#133
Merged
Merged
Conversation
hannahli-nv
added a commit
that referenced
this pull request
May 26, 2026
nvskills-ci only watches the 'skills/' prefix (along with team-skills/, rules/team-rules/, plugins/). Our skills lived at .agents/skills/, so /nvskills-ci would be skipped with "no supported changes". Move the 7 cuTile skill folders to skills/. Keep .agents/skills and .claude/skills as symlinks to ../skills for backward compatibility (Claude Code skill loader, agent-tool wrappers). Also: - LICENSE: add skills/ to dual-license scope alongside the symlinks. - CONTRIBUTING.md: reference skills/ in the DCO section. - check_spdx_headers.py: walker targets skills/; add to SKIP_DIRS. Companion PR #133 adds the .github/workflows/request-nvskills-ci.yml trigger workflow. Signed-off-by: Hannah Li <hanli@nvidia.com>
Collaborator
Author
|
/ok to test 2a9ee7e |
xjmxyt
approved these changes
May 26, 2026
hannahli-nv
force-pushed
the
add-nvskills-ci-workflow
branch
from
2a9ee7e to
615c5cd
Compare
Collaborator
Author
|
/ok to test 615c5cd |
Companion to PR #132 which migrates the skills/ canonical path. Signed-off-by: Hannah Li <hanli@nvidia.com>
hannahli-nv
force-pushed
the
add-nvskills-ci-workflow
branch
from
615c5cd to
cac740f
Compare
Collaborator
Author
|
/ok to test cac740f |
hannahli-nv
added a commit
that referenced
this pull request
May 26, 2026
nvskills-ci only watches the 'skills/' prefix (along with team-skills/, rules/team-rules/, plugins/). Our skills lived at .agents/skills/, so /nvskills-ci would be skipped with "no supported changes". Move the 7 cuTile skill folders to skills/. Keep .agents/skills and .claude/skills as symlinks to ../skills for backward compatibility (Claude Code skill loader, agent-tool wrappers). Also: - LICENSE: add skills/ to dual-license scope alongside the symlinks. - CONTRIBUTING.md: reference skills/ in the DCO section. - check_spdx_headers.py: walker targets skills/; add to SKIP_DIRS. Companion PR #133 adds the .github/workflows/request-nvskills-ci.yml trigger workflow. Signed-off-by: Hannah Li <hanli@nvidia.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add
.github/workflows/request-nvskills-ci.yml— the trigger workflow used by nvskills-ci.What it does
The workflow fires under two conditions:
/nvskills-cion a pull request.nv-skills-ci[bot]pushes a commit titled "Attach NVSkills validation signatures" (follow-up verification path).In both cases it calls the central reusable workflow which validates the request, then dispatches to the downstream signing pipeline.
Why a separate PR
issue_comment-triggered workflows must already exist on the default branch before GitHub will fire them. Landing this small standalone PR first lets the next content PR (#132 —skills/path migration) be the one that exercises/nvskills-ciend to end.Follow-up (separate from this PR)
NVSKILLS_CI_DISPATCH_TOKENto be added to repo secrets.NVSKILLS_SIGNATURE_PUSH_ACTORrepo variable to be set tonv-skills-ci[bot].CI Configuration