A small update on running tests docs

[Kipok]

Summary by CodeRabbit

• Documentation

  • Updated contributing guide with clarified testing instructions, including guidance on running CPU and GPU tests and the proper pytest invocation flags.
  • Refined sign-off contribution guidance.

• Chores

  • Updated configuration and ignore settings.

[greptile-apps]

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

Incorrect pytest flag

This says to add pytest’s -s flag, but -s is a git commit -s/--signoff convention and in pytest it disables output capture (not generally required for tests to pass). If the intent is to avoid failures due to missing stdout capture, that’s unlikely; if the intent is to stop at first failure or increase verbosity, the flags should be -x / -vv. As written, contributors may run the wrong command and still get failing tests.

Consider replacing this with the exact pytest ... invocation you expect contributors to run (including any required env vars), or remove the -s guidance unless it’s truly required for correctness.

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request contains configuration and documentation updates. The CodeRabbit auto-review flag is enabled for incremental reviews, a markdown file is added to git ignore, and contribution guidelines are expanded with testing instructions and clarified sign-off procedures.

Changes

Cohort / File(s)	Summary
Configuration Files .coderabbit.yaml, .gitignore	Enabled automatic incremental code reviews and added AGENTS.md to the ignore list.
Documentation CONTRIBUTING.md	Expanded testing guidance with references to CI tests and Slurm documentation; clarified pytest flag requirements for CPU tests and refined sign-off branch guidance.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: updates to the CONTRIBUTING.md documentation regarding running tests, which is the most substantive change in the PR.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch igitman/test-doc

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[greptile-apps]

_{No files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

_{No files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@CONTRIBUTING.md`:
- Line 166: Update the sentence to use the proper noun "GitHub Actions" instead
of "github actions" and change the link text to be descriptive by replacing
"Slurm tests documentation is [here](/tests/slurm-tests)" with "Slurm tests
documentation" as the link text (e.g., "Check existing GitHub Actions CI for
cpu/gpu tests. Slurm tests documentation: [Slurm tests
documentation](/tests/slurm-tests)").

🧹 Nitpick comments (1)

CONTRIBUTING.md (1)

170-170: "More details TBD" placeholder left in the docs.

Consider either fleshing this out before merging or converting it to a tracked TODO/issue so it doesn't get forgotten.

Would you like me to open an issue to track completing this section?

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Check if /tests/slurm-tests exists as a directory or file
fd -t d 'slurm-tests' tests/ 2>/dev/null || echo "No directory named slurm-tests found"
fd -t f 'slurm-tests*' tests/ 2>/dev/null || echo "No files matching slurm-tests* found"
# Also check for any README or index within tests directory
find tests/ -maxdepth 2 -name "*slurm*" 2>/dev/null || echo "No slurm-related files found in tests/"
# List what's actually in tests/ directory to understand structure
ls -la tests/ 2>/dev/null | head -20

Repository: NVIDIA-NeMo/Skills

Length of output: 1373

---

Capitalize "GitHub" and use descriptive link text.

• "github actions" → "GitHub Actions" (proper noun).
• Replace non-descriptive link text with "Slurm tests documentation".

📝 Proposed fix

-Check existing github actions CI for cpu/gpu tests. Slurm tests documentation is [here](/tests/slurm-tests).
+Check existing GitHub Actions CI for cpu/gpu tests. See the [Slurm tests documentation](/tests/slurm-tests).

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	Check existing github actions CI for cpu/gpu tests. Slurm tests documentation is [here](/tests/slurm-tests).
	Check existing GitHub Actions CI for cpu/gpu tests. See the [Slurm tests documentation](/tests/slurm-tests).

🧰 Tools

🪛 LanguageTool

[uncategorized] ~166-~166: The official name of this software platform is spelled with a capital “H”.
Context: ... ``` ## Running Tests Check existing github actions CI for cpu/gpu tests. Slurm tes...

(GITHUB)

🪛 markdownlint-cli2 (0.20.0)

[warning] 166-166: Link text should be descriptive

(MD059, descriptive-link-text)

🤖 Prompt for AI Agents

In `@CONTRIBUTING.md` at line 166, Update the sentence to use the proper noun
"GitHub Actions" instead of "github actions" and change the link text to be
descriptive by replacing "Slurm tests documentation is
[here](/tests/slurm-tests)" with "Slurm tests documentation" as the link text
(e.g., "Check existing GitHub Actions CI for cpu/gpu tests. Slurm tests
documentation: [Slurm tests documentation](/tests/slurm-tests)").