Skip to content

Add SECURITY.md, fix workflow permissions, redesign README#131

Merged
sebastientaggart merged 1 commit into
devfrom
feature/github-polish
Apr 12, 2026
Merged

Add SECURITY.md, fix workflow permissions, redesign README#131
sebastientaggart merged 1 commit into
devfrom
feature/github-polish

Conversation

@sebastientaggart

Copy link
Copy Markdown
Member

Polish the project's GitHub presence with four changes:

  • SECURITY.md: Add a lightweight security policy with GitHub Issues reporting and a private-disclosure carve-out for supply-chain concerns
  • Workflow permissions: Add explicit permissions: { contents: read } to both sync-check.yml and test.yml, resolving the CodeQL "workflow does not contain permissions" warnings
  • README redesign: Restructure the header with centered title, tagline, badge row, and hero image (LiteLLM-style layout) while preserving all existing content

Issue #130

Add a lightweight security policy, restrict GITHUB_TOKEN permissions
to contents:read in both CI workflows to resolve CodeQL warnings,
and restructure the README with a centered header/tagline/badge layout.
@sebastientaggart

Copy link
Copy Markdown
Member Author

Review Summary

Verdict: APPROVE

Findings

No issues found. Code looks correct and follows project conventions.

  • The permissions: { contents: read } additions to both workflow files correctly apply least-privilege and resolve the CodeQL warnings.
  • SECURITY.md is reasonable for the project scope and correctly distinguishes public vs. private disclosure paths.
  • README restructure preserves all existing badge links and content; only layout changes.

@sebastientaggart
sebastientaggart merged commit ae76615 into dev Apr 12, 2026
6 checks passed
@sebastientaggart
sebastientaggart deleted the feature/github-polish branch April 12, 2026 17:14
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.

1 participant