Mcore linting and formatting

Skill NVIDIA/skills/skills/mcore-linting-and-formatting

Linting and formatting for Megatron-LM. Covers running autoformat.sh, tools (ruff, black, isort, pylint, mypy), and code style rules.From its SKILL.md

Install

npx -y skills add NVIDIA/skills --skill mcore-linting-and-formatting

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

What its file declares

Copied from the file, not written here

The file declares its own license as Apache-2.0. That is the author’s claim about this one file, and it is not the same thing as the license GitHub reports for the repository, which is listed with the other numbers below.

SKILL.md

1.6 KB, 306 tokens by cl100k_base, as published. Nobody here has run it

Linting and Formatting

Running the Formatter

Run before opening a PR:

# Check mode (no changes applied)
BASE_REF=main CHECK_ONLY=true SKIP_DOCS=false bash tools/autoformat.sh

# Fix mode
BASE_REF=main CHECK_ONLY=false bash tools/autoformat.sh

Tools invoked: black, isort, pylint, ruff, mypy.

Import Ordering

After editing imports in any Python files, always run uv run isort on those files before committing:

uv run isort <file1>.py <file2>.py

Setting Up the Linting Group

Inside the container:

uv sync --locked --only-group linting

This installs ruff, black, isort, pylint — the same tools used by tools/autoformat.sh and CI's linting job.

Code Style Rules

Type hints: required on all public API functions. Use X | None, not Optional[X].
Docstrings: Google-style on all public classes and functions.
Naming: follow Python conventions — snake_case for functions and variables, PascalCase for classes.
Line length: 119 characters (configured in pyproject.toml).
No bare except: always catch specific exception types.

What ships with it: 4 files

9.8 KB alongside SKILL.md

evals/

evals.json3 B