Skip to content

Conversation

@billwallis
Copy link
Owner

@billwallis billwallis commented Mar 11, 2025

Summary by Sourcery

Migrates the project from Poetry to UV for dependency management, updating the build system, CI configuration, pre-commit hooks, and documentation accordingly.

Enhancements:

  • Replaces Poetry with UV for dependency management.
  • Updates the project to use setuptools as a build backend.
  • Updates dependencies to their latest versions, and groups them into development, documentation, and testing categories.
  • Updates the documentation to reflect the change to UV.
  • Removes the poetry.lock file and adds a uv.lock file.
  • Updates the pre-commit configuration to use uv-lock instead of poetry-lock.
  • Updates the mkdocs configuration to remove the tags file.

@sourcery-ai

This comment was marked as resolved.

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey @Bilbottom - I've reviewed your changes - here's some feedback:

Overall Comments:

  • Consider adding a uv venv step to the workflow to create a virtual environment before syncing dependencies.
  • Update the README to reflect the change from Poetry to uv for dependency management.
Here's what I looked at during the review
  • 🟢 General issues: all looks good
  • 🟢 Security: all looks good
  • 🟢 Review instructions: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@billwallis billwallis merged commit c5d9e1c into main Mar 11, 2025
2 checks passed
@billwallis billwallis deleted the switch-to-uv branch March 11, 2025 21:39
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