Skip to content
This repository has been archived by the owner on Nov 29, 2024. It is now read-only.

Migrate app to use uv for dependencies #368

Merged
merged 3 commits into from
Nov 13, 2024
Merged

Conversation

samuelhwilliams
Copy link
Contributor

Ticket: https://mhclgdigital.atlassian.net/browse/FSPT-73

Change description

This migrates us from using pip/pip-tools to using uv to manage dependencies. The main updates are:

  • Removing requirements*.txt/in files and migrating their contents to pyproject.toml.
  • Using a new uv.lock file to pin versions/hashes.
  • Updating the Dockerfile to take a consistent approach to local development.

How to test

  • Deploy this branch to dev and run e2e tests
  • Checkout all the uv app/docker-runner branches and run it locally.

While also maintaining the existing pinned versions of requirements from requirements.txt

Patch autogenerated by funding-service-design-docker-runner/scripts/migrate-apps-to-uv.sh
Copy link

@MarcUsher MarcUsher left a comment

Choose a reason for hiding this comment

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

This looks good to me 👍

@samuelhwilliams samuelhwilliams merged commit 4491abc into main Nov 13, 2024
14 checks passed
@samuelhwilliams samuelhwilliams deleted the uv-requirements branch November 13, 2024 13:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants