Skip to content

docs: add SkillsLLM security check badge to README - #899

Merged
vitali87 merged 1 commit into
mainfrom
docs/skillsllm-security-badge
Jul 22, 2026
Merged

docs: add SkillsLLM security check badge to README#899
vitali87 merged 1 commit into
mainfrom
docs/skillsllm-security-badge

Conversation

@vitali87

@vitali87 vitali87 commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Adds the SkillsLLM security check badge to the README badge row, linking to the repository's security assessment at https://skillsllm.com/security-check/DHsMGRb1Ysys. The badge SVG endpoint was verified to return 200 with image/svg+xml.

Summary by CodeRabbit

  • Documentation
    • Added a SkillsLLM Security Check badge and link to the README header alongside the existing badges.

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: e52920ca-8dc6-4607-951a-fd10ff49aa77

📥 Commits

Reviewing files that changed from the base of the PR and between fb48ac3 and f680167.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

The README header now includes a SkillsLLM security check badge linked to the corresponding security check.

Changes

README badge update

Layer / File(s) Summary
Add security check badge
README.md
Adds a linked SkillsLLM security check badge beside the existing header badges.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: zc277584121

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is on-topic but does not follow the required template and omits Summary, Type of Change, Related Issues, Test Plan, and Checklist sections. Rewrite the PR description using the provided template and fill in each required section, especially Summary, Type of Change, Test Plan, and Checklist.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and clearly summarizes the README badge addition.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/skillsllm-security-badge

Comment @coderabbitai help to get the list of available commands.

@vitali87

Copy link
Copy Markdown
Owner Author

@greptile review

@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds a SkillsLLM security-check badge to the README. The main changes are:

  • Links the badge to the repository’s security assessment.
  • Adds the remote SVG to the existing badge row.

Confidence Score: 5/5

This looks safe to merge.

No blocking issues found in the changed code.

T-Rex T-Rex Logs

What T-Rex did

  • T-Rex validated the general contract, confirming the initial state where the README at HEAD^ lacked both SkillsLLM URLs and the external endpoints were reachable.
  • T-Rex verified the after-state by confirming that the exact markup assertion passed and both endpoints returned 200 OK.
  • T-Rex noted that the fetched SVGs were preserved as artifacts and identified as valid SVG files.

View all artifacts

T-Rex Ran code and verified through T-Rex

Important Files Changed

Filename Overview
README.md Adds a valid linked security badge using the repository’s existing owner and name.

Reviews (1): Last reviewed commit: "docs: add SkillsLLM security check badge..." | Re-trigger Greptile

@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds a SkillsLLM security-check badge to the README. The main changes are:

  • Adds the security assessment badge to the existing badge row.
  • Links the badge to the repository’s SkillsLLM assessment.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

T-Rex T-Rex Logs

What T-Rex did

  • Captured the pre-change README assertions and live endpoint responses to establish the baseline.
  • Captured the post-change README additions, full HTTP status and header evidence, and a bounded SVG body sample for comparison.
  • Linked and reviewed the two log artifacts to support the pre-change and post-change evidence.

View all artifacts

T-Rex Ran code and verified through T-Rex

Important Files Changed

Filename Overview
README.md Adds a linked SkillsLLM security badge using the repository’s canonical owner and name.

Reviews (1): Last reviewed commit: "docs: add SkillsLLM security check badge..." | Re-trigger Greptile

@sonarqubecloud

Copy link
Copy Markdown

@vitali87
vitali87 merged commit 0c55a3b into main Jul 22, 2026
24 checks passed
@vitali87
vitali87 deleted the docs/skillsllm-security-badge branch July 22, 2026 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant