docs: list nemoclaw user agent skills - #5398
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughUpdates ChangesAgent Skills Documentation
Estimated Code Review Effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Signed-off-by: RlxChap2 <mmah02778@gmail.com>
1aa8036 to
2039e4d
Compare
|
Thanks for the docs fix. Before this can move forward, please read the NemoClaw contributing guide and make sure the PR satisfies the DCO/sign-off requirements:
After that, please confirm here that you read the contributing guide and that DCO is passing. |
|
Update: this may have been a temporary CI failure. All checks are now passing, so I am proceeding with merge review. |
|
Correction: DCO and the status checks are passing, but the PR is still blocked by the verified-signature requirement. The remaining issue is commit |
miyoungc
left a comment
There was a problem hiding this comment.
Requesting changes because this PR contains an unverified commit signature.
Repository policy requires PRs to satisfy both gates before merge:
- DCO must pass for every commit.
- GitHub must show every commit as verified.
DCO is currently passing here, but commit 2039e4d is unsigned/unverified. Please amend or recreate the contributor commit with a verified GitHub commit signature and push the updated branch. PRs with unverified commits or failing DCO cannot be accepted.
|
Correction on the remediation path: because force-push is not allowed, this PR cannot be fixed in place if the unsigned commit remains in the branch history. The required path is to open a fresh PR with a clean commit history where every commit is both DCO-signed and GitHub-verified. We cannot accept this PR while it contains the unverified |
should i open a new PR ? |
|
i'm gonna open a new PR rn |
|
Opened a fresh replacement PR with a clean one-commit history: #5522 The new commit includes the DCO sign-off and GitHub reports its signature as verified. I did not force-push the original branch. |
|
Thank you for reopening a fresh PR. Closing this PR. |
## Summary Adds the missing `nemoclaw-user-agent-skills` entry to the Agent Skills documentation so the page lists all user skills shipped under `.agents/skills`. ## Related Issue Fixes #5083 ## Replacement PR Replaces #5398 with a clean one-commit branch because the original PR history contained an unverified commit. ## Changes - Added `nemoclaw-user-agent-skills` to the Available Skills table. - Added an example question that maps to `nemoclaw-user-agent-skills`. ## Type of Change - [ ] Code change (feature, bug fix, or refactor) - [ ] Code change with doc updates - [x] Doc only (prose changes, no code sample modifications) - [ ] Doc only (includes code sample changes) ## Verification - [x] DCO trailer is present on the commit. - [x] GitHub reports the commit signature as verified. - [x] No secrets, API keys, or credentials committed. - [x] Docs updated for user-facing behavior changes. - [ ] `npm run docs` builds without warnings (doc changes only) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Documentation * Expanded skills documentation with information on NemoClaw's agent skills, including setup guidance and example question mappings for coding assistant integration. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Signed-off-by: RlxChap2 <mmah02778@gmail.com> Signed-off-by: RlxChap2 <mmah02778@gmail.com> Co-authored-by: Miyoung Choi <miyoungc@nvidia.com>
Summary
Adds the missing
nemoclaw-user-agent-skillsentry to the Agent Skills documentation so the page lists all user skills shipped under.agents/skills.Related Issue
Fixes #5083
Changes
nemoclaw-user-agent-skillsto the Available Skills table.nemoclaw-user-agent-skills.Type of Change
Verification
npx prek run --from-ref main --to-ref HEADpassesnpm testpasses (broad runtime changes only)npm run docsbuilds without warnings (doc changes only)Signed-off-by: RlxChap2 mmah02778@gmail.com
Summary by CodeRabbit