Menú
Contribution conventions for NeMo-RL. Covers PR title format, commit sign-off, and CI triggering.
Instalar con Codex o Claude Copia este prompt, pégalo en Codex, Claude u otro asistente, y deja que revise la página de la skill y la instale por ti.
Basado en la clasificación ocupacional SOC
Configuration conventions for NeMo-RL. YAML is the single source of truth for defaults. Covers BaseModel/TypedDict usage, dataclass for internal classes, exemplar YAML updates, and forbidden default patterns.
Error handling guidelines for NeMo-RL. Covers exception specificity, minimal try bodies, and else blocks.
Code style guidelines for NeMo-RL (Python and shell). Covers naming, indentation, comments, docstrings, reflection avoidance, and uv usage.
Interactive code review for NVIDIA-NeMo/RL pull requests. Checks out PR locally, reads existing comments, applies coding guidelines from skills, previews findings, and posts review comments. Also supports reviewing the current branch locally.
Build and dependency management for NeMo-RL. Covers Docker image building and running, uv usage, venv setup, and adding dependencies.
CI/CD reference for NeMo-RL. Covers GitHub Actions pipeline structure, CI triggering via /ok to test, and CI failure investigation.
| name | contributing |
| description | Contribution conventions for NeMo-RL. Covers PR title format, commit sign-off, and CI triggering. |
| when_to_use | Opening a PR; writing a commit message; triggering CI; 'PR title format', 'sign-off', 'conventional commits', 'how do I trigger CI', during code review of PR process. |
PR titles must follow the Conventional Commits spec. This is enforced by the semantic-pull-request CI check.
<type>[optional scope]: <description>
Allowed types:
| Type | When to use |
|---|---|
feat | New feature |
fix | Bug fix |
ci | CI/CD changes |
docs | Documentation only |
refactor | Code restructuring without behaviour change |
test | Adding or fixing tests |
chore | Maintenance (deps, configs, tooling) |
perf | Performance improvement |
build | Build system changes |
revert | Reverts a previous commit |
Do:
ci: retry apt-get installs to handle mirror sync failures
feat(grpo): add dataclass config defaults infrastructure
fix: preserve RAY_EXPERIMENTAL_NOSET_CUDA_VISIBLE_DEVICES
Don't:
[ci] fix: retry apt-get installs ← area tags are not part of this convention
Update stuff
Fix bug
All commits must be signed off with -s:
git commit -s -m "fix: correct reward normalization"
After pushing, trigger CI with:
/ok to test <full-commit-sha>
Use git rev-parse HEAD (not the short form) to get the full SHA.