fix(ci): add sqlalchemy to pip install step (closes #293) #300

Closed
hongming-pc2 wants to merge 1 commits from ci/fix-293-sqlalchemy-pip-install into main
Owner

Summary

  • Add sqlalchemy to the pip install step in .github/workflows/ci.yml
  • test_audit_ledger.py imports sqlalchemy directly; adding it explicitly ensures CI collects this test reliably

Closes #293

🤖 Generated with Claude Code

## Summary - Add `sqlalchemy` to the pip install step in `.github/workflows/ci.yml` - `test_audit_ledger.py` imports sqlalchemy directly; adding it explicitly ensures CI collects this test reliably Closes #293 🤖 Generated with [Claude Code](https://claude.com/claude-code)
hongming-pc2 added 1 commit 2026-05-10 11:59:16 +00:00
fix(ci): add sqlalchemy to pip install step (closes #293)
Some checks failed
Secret scan / Scan diff for credential-shaped strings (pull_request) Failing after 2s
sop-tier-check / tier-check (pull_request) Failing after 3s
audit-force-merge / audit (pull_request) Has been skipped
89f6766e2c
test_audit_ledger.py imports sqlalchemy directly. While sqlalchemy
is declared in workspace/requirements.txt, add it explicitly to the
pip install step to ensure CI collects this test reliably even if
requirements.txt has parsing issues.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
hongming-pc2 closed this pull request 2026-05-10 12:10:58 +00:00
Some checks are pending
Secret scan / Scan diff for credential-shaped strings (pull_request) Failing after 2s
sop-tier-check / tier-check (pull_request) Failing after 3s
audit-force-merge / audit (pull_request) Has been skipped
CI / all-required (pull_request)
Required
sop-checklist / all-items-acked (pull_request)
Required

Pull request closed

Sign in to join this conversation.
No description provided.