Skip to content

feat: add linting, type checking, README badges, and Makefile#18

Merged
amahpour merged 1 commit intomainfrom
17-add-readme-badges-lint-and-static-analysis
Mar 30, 2026
Merged

feat: add linting, type checking, README badges, and Makefile#18
amahpour merged 1 commit intomainfrom
17-add-readme-badges-lint-and-static-analysis

Conversation

@amahpour
Copy link
Copy Markdown
Owner

@amahpour amahpour commented Mar 30, 2026

Summary

Closes #17

  • Add ruff linter and formatter with CI job (lint + format check)
  • Add mypy static type checker with CI job
  • Fix all existing lint and type errors across the codebase
  • Add README badges: CI status, Python version, License, Ruff, mypy
  • Add Makefile with help, up, down, test, lint, format, typecheck, check, clean targets
  • CI now also triggers on push to main (required for status badge)

Test plan

  • ruff check . passes
  • ruff format --check . passes
  • mypy server/ passes
  • All 255 unit tests pass
  • CI pipeline passes with new lint and type-check jobs

🤖 Generated with Claude Code

Add ruff and mypy to CI pipeline, fix all existing lint and type errors,
add project status badges to README, and create a Makefile for common
dev workflows.

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

Coverage

Unit Test Coverage
FileStmtsMissCoverMissing
__init__.py00100% 
db.py133298%131–132
hooks.py138298%175, 178
main.py370100% 
pr_lookup.py113298%53–54
sessions.py00100% 
terminal.py980100% 
watcher.py3801695%25–28, 337, 528–529, 563–571
routes
   __init__.py00100% 
   api.py1460100% 
   ws.py106892%64–65, 103–104, 106, 125, 141–142
TOTAL11513097% 

Tests Skipped Failures Errors Time
255 0 💤 0 ❌ 0 🔥 16.821s ⏱️

@amahpour amahpour merged commit 3e5cb13 into main Mar 30, 2026
6 checks passed
@amahpour amahpour deleted the 17-add-readme-badges-lint-and-static-analysis branch March 30, 2026 02:58
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 README badges, CI lint stage, and static analysis

1 participant