Skip to content

add: uv - #14

Merged
moonorange merged 2 commits into
mainfrom
13-use-uv-for-dependency-and-environment-management
Mar 1, 2026
Merged

moonorange merged 2 commits into
mainfrom
13-use-uv-for-dependency-and-environment-management

Conversation

@moonorange

@moonorange moonorange commented Feb 28, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Switch dependency and environment management to uv: single tool for install, lock, and venv.

Changes

  • pyproject.toml: Project metadata and dependencies (replacing requirements.txt). Optional dev extra for ruff, pyright, coverage.
  • uv.lock: Lockfile for reproducible installs; generated by uv lock / uv sync.
  • .python-version: Pinned to 3.12 (ChromaDB/Pydantic v1 incompatible with 3.14+).
  • README: Installation and run instructions use uv sync --extra dev, venv activation (source .venv/bin/activate), and normal python commands; uv run documented as alternative when venv is not activated.
  • github_downloader/README.md: Aligned with main README (venv activation, then python -m github_downloader).

Developer workflow

  • Install: uv sync --extra dev then activate .venv to use python / pytest / make without prefixing with uv run.
  • Re-lock when deps change: uv lock (or uv sync will re-lock when needed).

@moonorange moonorange linked an issue Feb 28, 2026 that may be closed by this pull request
@moonorange
moonorange force-pushed the 13-use-uv-for-dependency-and-environment-management branch 3 times, most recently from db4f961 to 45106f0 Compare February 28, 2026 15:29
Signed-off-by: moonorange <keigokida51@gmail.com>
@moonorange
moonorange force-pushed the 13-use-uv-for-dependency-and-environment-management branch from 45106f0 to 33c118b Compare February 28, 2026 15:33
ArchCipher
ArchCipher previously approved these changes Mar 1, 2026

@ArchCipher ArchCipher left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check my comment on README.md changes

Comment thread README.md Outdated
With the venv activated (see Installation), from the repo root:

- Run programmatically: `python chatbot.py`
- Run via CLI: `fastapi dev chatbot.py` or `uvicorn chatbot:app --reload`

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe just say
run: python chatbot.py or fastapi dev chatbot.py or uvicorn chatbot:app --reload
no need to mention via CLI only for the second, when both are run via CLI

@moonorange moonorange Mar 1, 2026 •

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed
5321501

Signed-off-by: moonorange <keigokida51@gmail.com>
@moonorange
moonorange merged commit d06b7c9 into main Mar 1, 2026
1 check passed
@moonorange
moonorange deleted the 13-use-uv-for-dependency-and-environment-management branch March 1, 2026 17:18
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.

Use uv for dependency and environment management

2 participants