Skip to content

docs(readme): add HVTrust badge - #428

Open
deepujain wants to merge 1 commit into
NVIDIA:mainfrom
deepujain:docs/277-hvtrust-badge
Open

docs(readme): add HVTrust badge#428
deepujain wants to merge 1 commit into
NVIDIA:mainfrom
deepujain:docs/277-hvtrust-badge

Conversation

@deepujain

Copy link
Copy Markdown
Contributor

Summary

  • add the requested auto-updating HVTrust badge beside the existing README trust badges
  • link the badge to SkillSpector's public HVTracker evidence page

Validation

  • git diff --check — clean
  • verified the badge endpoint returns HTTP 200 with image/svg+xml
  • verified the destination page returns HTTP 200 and identifies SkillSpector
  • rendered the live SVG locally and confirmed the badge is legible

Risk

  • documentation-only change; no scanner or package behavior changes
  • the score is served dynamically by the third-party badge endpoint

Fixes #277

@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.

[SkillSpector Review]\n\nApproved. This is a documentation-only badge addition with a valid target and no code-path impact. Current applicable checks pass.

@deepujain
deepujain force-pushed the docs/277-hvtrust-badge branch 2 times, most recently from 6a8d02e to b889a13 Compare August 24, 2026 18:17
@rng1995
rng1995 enabled auto-merge (squash) August 24, 2026 18:27
Signed-off-by: Deepak Jain <deepujain@gmail.com>
auto-merge was automatically disabled August 24, 2026 18:54

Head branch was pushed to by a user without write access

@deepujain
deepujain force-pushed the docs/277-hvtrust-badge branch from a89c9d6 to f3a97eb Compare August 24, 2026 18:54
@deepujain

Copy link
Copy Markdown
Contributor Author

Rebased the documentation-only change onto current main at signed head f3a97eb. The badge SVG and target page both return HTTP 200, git diff --check is clean, and GitHub checks are rerunning. Ready for re-review.

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.

Add your HVTrust badge to the README?

2 participants