Skip to content

docs: fix provider registry path#30

Closed
CooperSheroy wants to merge 1 commit into
NVIDIA:mainfrom
CooperSheroy:catalina/docs-provider-registry-path-20260612
Closed

docs: fix provider registry path#30
CooperSheroy wants to merge 1 commit into
NVIDIA:mainfrom
CooperSheroy:catalina/docs-provider-registry-path-20260612

Conversation

@CooperSheroy

@CooperSheroy CooperSheroy commented Jun 12, 2026

Copy link
Copy Markdown

Summary

  • Fix the documented bundled provider registry path to match the package layout: src/skillspector/providers/{provider}/model_registry.yaml.
  • Canonicalize README and package metadata GitHub links to NVIDIA/SkillSpector.

Validation

  • python3.14 parsed pyproject.toml successfully.
  • git diff --check passed.
  • Verified canonical GitHub URLs return HTTP 200.

Signed-off-by: CooperSheroy <sheroycoops@gmail.com>
@Credok12

Copy link
Copy Markdown

Ok

@keshprad

Copy link
Copy Markdown
Member

Closing in favor of #79, which includes this provider-registry documentation fix in the current OSS release snapshot.

@keshprad keshprad closed this Jun 16, 2026
keshprad added a commit that referenced this pull request Jun 16, 2026
Refresh the public SkillSpector snapshot from the internal OSS release branch
release/oss-2026-06-15 at a7ebb82733fba54512c546fed08cd17a2cbc4a17.

Changes:
- Correct the documented bundled model registry path for SKILLSPECTOR_MODEL_REGISTRY.
- Add explicit returns in Docker smoke helper functions.

Notes:
- Rebased onto latest GitHub main after #78 merged.
- Supersedes the original provider-registry documentation PR #30.

Verification:
- PATH="$PWD/.venv/bin:$PATH" ./scripts/create-oss-release.sh release/oss-2026-06-15
  - includes make test-unit: 600 passed, 11 skipped, 22 deselected
- git diff --check origin/main..HEAD
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.

3 participants