Skip to content

skills: rename resources/ to references/#1242

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

skills: rename resources/ to references/#1242
rgsl888prabhu wants to merge 2 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: 8eb14194-3786-47d7-ad3f-5dd5111def40

📥 Commits

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

📒 Files selected for processing (27)
  • .github/workflows/pr.yaml
  • skills/cuopt-developer/SKILL.md
  • skills/cuopt-developer/evals/evals.json
  • skills/cuopt-developer/references/build_and_test.md
  • skills/cuopt-developer/references/contributing.md
  • skills/cuopt-developer/references/conventions.md
  • skills/cuopt-developer/references/first_time_setup.md
  • skills/cuopt-developer/references/python_bindings.md
  • skills/cuopt-developer/references/troubleshooting.md
  • skills/cuopt-install/SKILL.md
  • skills/cuopt-install/references/verification_examples.md
  • skills/cuopt-numerical-optimization-api-c/SKILL.md
  • skills/cuopt-numerical-optimization-api-c/assets/lp_basic/README.md
  • skills/cuopt-numerical-optimization-api-c/assets/lp_duals/README.md
  • skills/cuopt-numerical-optimization-api-c/assets/milp_basic/README.md
  • skills/cuopt-numerical-optimization-api-c/assets/milp_production_planning/README.md
  • skills/cuopt-numerical-optimization-api-c/assets/mps_solver/README.md
  • skills/cuopt-numerical-optimization-api-c/references/examples.md
  • skills/cuopt-numerical-optimization-api-python/SKILL.md
  • skills/cuopt-numerical-optimization-api-python/assets/README.md
  • skills/cuopt-numerical-optimization-api-python/references/qp_examples.md
  • skills/cuopt-routing-api-python/SKILL.md
  • skills/cuopt-routing-api-python/assets/README.md
  • skills/cuopt-routing-api-python/assets/pdp_basic/README.md
  • skills/cuopt-routing-api-python/assets/vrp_basic/README.md
  • skills/cuopt-routing-api-python/references/examples.md
  • skills/cuopt-routing-api-python/references/server_examples.md

📝 Walkthrough

Walkthrough

This PR systematically migrates documentation reference paths across the codebase from resources/ to references/ directory structure. Changes span CI workflow exclusion patterns, developer skill guides, evaluation data, and example asset documentation, affecting six distinct skill areas.

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 all documentation links to references/ for first-time setup, build/test, Python bindings, contributing workflow, and troubleshooting guidance in both skill descriptor and evaluation scenarios.
Installation skill documentation
skills/cuopt-install/SKILL.md
Update verification_examples.md reference path to point to references/ directory.
C API skill documentation and examples
skills/cuopt-numerical-optimization-api-c/SKILL.md, skills/cuopt-numerical-optimization-api-c/assets/*/README.md
Update examples.md references across C API skill and all example asset READMEs (lp_basic, lp_duals, milp_basic, milp_production_planning, mps_solver).
Python numerical optimization API skill documentation
skills/cuopt-numerical-optimization-api-python/SKILL.md, skills/cuopt-numerical-optimization-api-python/assets/README.md
Update qp_examples.md references in both skill descriptor and assets README.
Python routing API skill documentation and examples
skills/cuopt-routing-api-python/SKILL.md, skills/cuopt-routing-api-python/assets/*/README.md
Update examples.md and server_examples.md references across Python routing API skill and example asset READMEs (pdp_basic, vrp_basic).

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the main change: renaming resources/ to references/ across skills directories.
Description check ✅ Passed The description is directly related to the changeset, explaining the rename from resources/ to references/ and noting it aligns with agentskills.io standard.
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
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