Skip to content

chore: repo hygiene — CodeQL, Dependabot, SECURITY.md, badges#12

Merged
nvandessel merged 4 commits into
mainfrom
chore/repo-hygiene
Mar 31, 2026
Merged

chore: repo hygiene — CodeQL, Dependabot, SECURITY.md, badges#12
nvandessel merged 4 commits into
mainfrom
chore/repo-hygiene

Conversation

@nvandessel
Copy link
Copy Markdown
Owner

Summary

  • CodeQL scanning workflow (weekly + on PRs)
  • Dependabot for pip and GitHub Actions dependencies
  • SECURITY.md with vulnerability reporting process
  • README badges: tests, CodeQL, license, Python version
  • Coverage reporting in CI
  • Repo description and topics set

No PyPI deployment

Not needed yet — hippofloop is used locally via uv run hippofloop. PyPI publishing can be added when there's a stable release.

nvandessel and others added 2 commits March 29, 2026 23:06
…ge in CI

- Add CodeQL scanning workflow (weekly + on PRs)
- Add Dependabot config for pip and GitHub Actions
- Add SECURITY.md with vulnerability reporting process
- Add README badges (tests, CodeQL, license, Python version)
- Add coverage reporting to CI test step
- Set repo description and topics

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Mar 29, 2026

Greptile Summary

This PR adds standard repo hygiene: CodeQL default-setup scanning, Dependabot for pip and GitHub Actions, a SECURITY.md, README badges, and coverage reporting in CI. All previous review concerns (broken CodeQL badge, misleading "Email:" label, inconsistent action pinning) have been addressed in prior commits.

  • .github/dependabot.yml — weekly updates for both pip and github-actions ecosystems, with sensible PR limits
  • .github/workflows/test.yml — coverage reporting added via --cov=hippofloop --cov-report=term-missing; pytest-cov>=5.0 is already declared in the dev extra in pyproject.toml, so no missing dependency
  • .gitignore.coverage correctly excluded to avoid committing coverage data
  • README.md — four badges added; the CodeQL badge URL uses the github-code-scanning/codeql path appropriate for GitHub's default setup (not a custom workflow file)
  • SECURITY.md — clean security policy pointing reporters to GitHub private security advisories with a 72-hour response SLA

Confidence Score: 5/5

Safe to merge — all changes are additive repo hygiene with no logic or runtime risk.

All prior review concerns have been resolved. No P0 or P1 issues found. Every change is either a new config file, a CI flag backed by an already-declared dependency, or a documentation update.

No files require special attention.

Important Files Changed

Filename Overview
.github/dependabot.yml New Dependabot config enabling weekly updates for pip and GitHub Actions dependencies — looks correct and well-configured.
.github/workflows/test.yml Adds pytest-cov flags to the test step; pytest-cov>=5.0 is already listed in dev dependencies in pyproject.toml, so no missing dependency issue.
.gitignore Adds .coverage to gitignore — correct companion to the new coverage reporting step.
README.md Adds four badges (Tests, CodeQL, License, Python); CodeQL badge now correctly points to github-code-scanning/codeql for the default setup workflow, addressing the prior broken-badge issue.
SECURITY.md New security policy file with private advisory link and 72-hour SLA; misleading 'Email:' label from earlier review has been removed.

Reviews (4): Last reviewed commit: "fix: CodeQL badge URL for default setup ..." | Re-trigger Greptile

Comment thread .github/workflows/codeql.yml Outdated
Comment thread SECURITY.md Outdated
- Remove custom codeql.yml — repo default setup already handles it
- Fix SECURITY.md misleading "Email:" label (Greptile P2)
- CodeQL badge points to code scanning page

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Comment thread README.md Outdated
@nvandessel nvandessel merged commit f06b4e4 into main Mar 31, 2026
5 checks passed
@nvandessel nvandessel deleted the chore/repo-hygiene branch March 31, 2026 04:45
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