Skip to content

chore: add Makefile, dev tooling, and GitHub Actions CI - #65

Merged
spChalk merged 10 commits into
mainfrom
chore/dev-tooling
Jul 21, 2026
Merged

chore: add Makefile, dev tooling, and GitHub Actions CI#65
spChalk merged 10 commits into
mainfrom
chore/dev-tooling

Conversation

@spChalk

@spChalk spChalk commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

What

  • Add GitHub Actions CI workflow (lint + test jobs, coverage artifacts, PR coverage comments)
  • Add Makefile with install, lint, lint-fix, test, clean, help targets
  • Configure Ruff (lint + format, line-length 120)
  • Configure Coverage (branch coverage, source = app/)
  • Add pre-commit hooks (ruff-check + ruff-format)
  • Restrict package to Python 3.12
  • Add tests/ scaffolding
  • Update README with make commands, CI badge, and Python 3.12-only note

Why

The repo had no automated dev workflow. This sets a consistent baseline for style, testing, and CI from the first commit.

Fixes

  • langfuse==4.64.14.1 (4.6 was never published; blocked uv sync)
  • Added hatchling config for the graphes-client git dependency and app/ package layout (both blocked uv sync)
  • ruff format now uses --check so CI fails on unformatted code

Verification

  • uv sync, make lint, make test pass locally
  • Workflow compiles and is pushed to chore/dev-tooling

@spChalk spChalk self-assigned this Jul 20, 2026
@spChalk
spChalk requested a review from Sbourgeat July 20, 2026 14:12

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

Praise: Good job!

Comment thread Makefile
Comment on lines +2 to +18

help:
@echo "Available targets:"
@echo " install-uv Install uv (if not already present)"
@echo " install-dev Install development dependencies"
@echo " install Install production dependencies"
@echo " lint Run ruff check and format diff"
@echo " lint-fix Run ruff check --fix and apply formatting"
@echo " test Run unit tests with coverage"
@echo " clean Remove test/coverage artifacts"

install-uv:
curl -LsSf https://astral.sh/uv/0.11.28/install.sh | sh

install-dev: install-uv
uv sync
uv run pre-commit install

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.

Praise: flowless 0 tests ran 🔥

@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown

☂️ Code Coverage

current status: ✅

Overall Coverage

Statements Covered Coverage Threshold Status
892 0 0% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
app/bots/admin/admin_bot.py 0% 🟢
app/bots/admin/cmi/cmi_bot.py 0% 🟢
app/bots/admin/cmi/cmi_restricted/cmi_restricted_bot.py 0% 🟢
app/bots/admin/lex/lex_bot.py 0% 🟢
app/bots/admin/plasma/plasma_bot.py 0% 🟢
app/bots/admin/sac/sac_bot.py 0% 🟢
app/bots/admin/servicedesk/servicedesk_bot.py 0% 🟢
app/bots/base.py 0% 🟢
app/bots/course/course_bot.py 0% 🟢
app/bots/course/debate/debate_bot.py 0% 🟢
app/bots/course/debate/micro452/micro452_bot.py 0% 🟢
app/bots/course/direct/env342/env342_bot.py 0% 🟢
app/bots/course/direct/math106e/math106e_bot.py 0% 🟢
app/bots/course/direct/statistics/statistics_bot.py 0% 🟢
app/bots/course/hinting/bioeng310/bioeng310_bot.py 0% 🟢
app/bots/course/hinting/math240/math240_bot.py 0% 🟢
app/bots/course/hinting/math261/math261_bot.py 0% 🟢
app/bots/course/hinting/math535/math535_bot.py 0% 🟢
app/bots/course/hinting/me331/me331_bot.py 0% 🟢
app/bots/course/hinting/micro315/micro315_bot.py 0% 🟢
app/bots/graph_chat/graph_chat_bot.py 0% 🟢
app/bots/graph_chat/graph_chat_tools/init.py 0% 🟢
app/bots/graph_chat/graph_chat_tools/exoset.py 0% 🟢
app/bots/graph_chat/graph_chat_tools/graph/clean.py 0% 🟢
app/bots/graph_chat/graph_chat_tools/graph/organisational_links.py 0% 🟢
app/bots/graph_chat/graph_chat_tools/graph/search.py 0% 🟢
app/bots/graph_chat/graph_chat_tools/news.py 0% 🟢
app/bots/main.py 0% 🟢
app/bots/nodes/classify.py 0% 🟢
app/bots/nodes/model.py 0% 🟢
app/bots/nodes/tools.py 0% 🟢
app/bots/prompts.py 0% 🟢
app/bots/registry.py 0% 🟢
app/config.py 0% 🟢
app/interfaces/graphai.py 0% 🟢
app/llms/utils.py 0% 🟢
app/logging_config.py 0% 🟢
app/main.py 0% 🟢
app/routers/public.py 0% 🟢
TOTAL 0% 🟢

updated for commit: 00d469e by action🐍

@spChalk spChalk changed the title chore: Add Makefile and dev tooling chore: add Makefile, dev tooling, and GitHub Actions CI Jul 21, 2026
@spChalk
spChalk merged commit a948815 into main Jul 21, 2026
2 checks passed
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.

2 participants