Skip to content

feat: support uv tool install and document in README#124

Merged
rng1995 merged 1 commit into
NVIDIA:mainfrom
tcconnally:feat/uv-tool-install-support
Jun 23, 2026
Merged

feat: support uv tool install and document in README#124
rng1995 merged 1 commit into
NVIDIA:mainfrom
tcconnally:feat/uv-tool-install-support

Conversation

@tcconnally

Copy link
Copy Markdown
Contributor

Summary

Fixes #99: Add explicit [tool.uv] section to pyproject.toml and document uv tool install as a quick installation method in the README.

Changes

  1. pyproject.toml: Add [tool.uv] section with comment documenting uv tool install support
  2. README.md: Add "Quick install with uv" section before the "From source" instructions

Usage

# One-command install (no clone needed)
uv tool install git+https://github.com/NVIDIA/skillspector.git

# Update to latest
uv tool update skillspector

Testing

All 621 tests pass. No code changes — packaging metadata only.

Add [tool.uv] section to pyproject.toml and document the
uv tool install quick-start command in the README. This
allows users to install SkillSpector with a single command:

  uv tool install git+https://github.com/NVIDIA/skillspector.git

Updates are equally simple: uv tool update skillspector

Fixes NVIDIA#99

Signed-off-by: Perseus Computing <51974392+tcconnally@users.noreply.github.com>

@rng1995 rng1995 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVE — adds a uv tool install quick-start to the README plus a [tool.uv] section, with no behavioral risk.

Minor / optional (non-blocking):

  • The [tool.uv] section is comment-only/empty, so it does not actually change anything — uv tool install git+... already works because the package defines a [project.scripts] entry point. The README docs are the valuable part; the empty section could be dropped, or a real setting added if one was intended.
  • Nit: the README uses a lowercased skillspector.git while the canonical repo name is SkillSpector; GitHub is case-insensitive here so it works, just flagging for consistency.

@rng1995 rng1995 merged commit 50e2b70 into NVIDIA:main Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

uv tool install

2 participants