Skip to content

feature/switch-from-using-uv-image-for-building#168

Merged
chopkinsmade merged 1 commit intomainfrom
feature/switch-from-using-uv-image-for-building
Feb 17, 2026
Merged

feature/switch-from-using-uv-image-for-building#168
chopkinsmade merged 1 commit intomainfrom
feature/switch-from-using-uv-image-for-building

Conversation

@chopkinsmade
Copy link
Contributor

@chopkinsmade chopkinsmade commented Feb 17, 2026

Description

Use a standard python image instead of the UV one for the initial build stage of our image. This gives greater control on what we can install, and removes a dependancy on UV keeping their image uptodate and patched. We still use the UV binary from their image for running the UV install

Instead of copying each file needed by the UV installer manually in the Dockerfile instructions, copy all files but use the .dockerignore to control the files that are copied

Contributors

Let's acknowledge the people who contributed to the work.

Type of change

  • Refactoring (made code better without changing its behaviour)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How this has been tested

Tested locally using the tests in the Makefile

Checklist

  • I have performed a self-review of my code
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

Reviewer Checklist

  • I have reviewed the PR and ensured no secret values are present

Signed-off-by: DBT pre-commit check
@chopkinsmade chopkinsmade force-pushed the feature/switch-from-using-uv-image-for-building branch from d3fe554 to 5fcac14 Compare February 17, 2026 11:05
@chopkinsmade chopkinsmade marked this pull request as ready for review February 17, 2026 11:08
@chopkinsmade chopkinsmade requested a review from a team as a code owner February 17, 2026 11:08
@chopkinsmade chopkinsmade merged commit 678fda1 into main Feb 17, 2026
18 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