Skip to content

skills: rename resources/ to references/#1242

Open
rgsl888prabhu wants to merge 4 commits into
NVIDIA:release/26.06from
rgsl888prabhu:skill-rename-resources-to-references
Open

skills: rename resources/ to references/#1242
rgsl888prabhu wants to merge 4 commits into
NVIDIA:release/26.06from
rgsl888prabhu:skill-rename-resources-to-references

Conversation

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

Align skill subfolder name with the agentskills.io standard. Pure rename — no content changes.

Align with the agentskills.io standard folder name. No content changes.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Ramakrishna Prabhu <ramakrishnap@nvidia.com>
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 19, 2026

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

Contributors can view more details about this message here.

@rgsl888prabhu rgsl888prabhu self-assigned this May 19, 2026
@rgsl888prabhu rgsl888prabhu added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels May 19, 2026
@rgsl888prabhu rgsl888prabhu marked this pull request as ready for review May 19, 2026 15:30
@rgsl888prabhu rgsl888prabhu requested review from a team as code owners May 19, 2026 15:30
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 19, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 18cd4d68-35b7-4df1-bdcc-5c7fc2c41d38

📥 Commits

Reviewing files that changed from the base of the PR and between e8248df and 52a9f57.

📒 Files selected for processing (3)
  • .github/workflows/pr.yaml
  • skills/cuopt-numerical-optimization-api-c/SKILL.md
  • skills/cuopt-routing-api-python/SKILL.md
✅ Files skipped from review due to trivial changes (2)
  • skills/cuopt-numerical-optimization-api-c/SKILL.md
  • skills/cuopt-routing-api-python/SKILL.md

📝 Walkthrough

Walkthrough

This PR migrates documentation links and CI changed-files exclusions from resources/... to references/... across workflow config, developer skill docs, evaluation data, install docs, C/Python skill READMEs, and routing assets.

Changes

Documentation Path Migration

Layer / File(s) Summary
CI workflow test exclusion patterns
.github/workflows/pr.yaml
Update changed-files exclusion patterns for three test groups (test_cpp, test_python_conda, test_python_wheels) from !skills/**/resources/** to !skills/**/references/**.
Developer skill documentation and evaluation data
skills/cuopt-developer/SKILL.md, skills/cuopt-developer/evals/evals.json
Update links to references/... for first-time setup, build/test, Python bindings, contributing workflow, and troubleshooting guidance in the developer skill descriptor and evaluation scenarios.
Installation skill documentation
skills/cuopt-install/SKILL.md
Change verification_examples.md link to point to references/verification_examples.md.
C API skill documentation and examples
skills/cuopt-numerical-optimization-api-c/SKILL.md, skills/cuopt-numerical-optimization-api-c/assets/*/README.md
Replace resources/examples.md with references/examples.md across C API SKILL and example READMEs (lp_basic, lp_duals, milp_basic, milp_production_planning, mps_solver).
Python numerical optimization API documentation
skills/cuopt-numerical-optimization-api-python/SKILL.md, skills/cuopt-numerical-optimization-api-python/assets/README.md
Point QP examples links to references/qp_examples.md.
Python routing API documentation and examples
skills/cuopt-routing-api-python/SKILL.md, skills/cuopt-routing-api-python/assets/*/README.md
Update examples and server_examples links to references/ equivalents in SKILL and asset READMEs (pdp_basic, vrp_basic).

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Possibly related PRs:

    • NVIDIA/cuopt#1249: Also modifies .github/workflows/pr.yaml changed-files exclusion patterns for skills paths.
    • NVIDIA/cuopt#1243: Modifies changed-files exclusion logic in the same workflow configuration.
  • Suggested reviewers:

    • Iroy30
    • tmckayus
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'skills: rename resources/ to references/' clearly and concisely describes the main change—renaming the resources directory to references across multiple skill files.
Description check ✅ Passed The description 'Align skill subfolder name with the agentskills.io standard. Pure rename — no content changes.' is directly related to the changeset, explaining the rationale and scope of the directory rename.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Member

@Iroy30 Iroy30 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 as long as we've got all the resources references.

@rgsl888prabhu rgsl888prabhu changed the base branch from main to release/26.06 May 20, 2026 17:28
@rgsl888prabhu
Copy link
Copy Markdown
Collaborator Author

/nvskills-ci

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator Author

/nvskills-ci

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants