Skip to content

backend: swap to a dependency lockfile#3305

Open
mistydemeo wants to merge 1 commit into
mainfrom
misty/locked_deps
Open

backend: swap to a dependency lockfile#3305
mistydemeo wants to merge 1 commit into
mainfrom
misty/locked_deps

Conversation

@mistydemeo
Copy link
Copy Markdown
Contributor

I chose to go with uv's lockfile format for convenience since that lets us track the dependency tree a little more easily. Happy to back that out and go for pip-compile or something else instead.

@socket-security
Copy link
Copy Markdown

socket-security Bot commented May 12, 2026

No dependency changes detected. Learn more about Socket for GitHub.

👍 No dependency changes detected in pull request

@mistydemeo
Copy link
Copy Markdown
Contributor Author

mistydemeo commented May 12, 2026

Alternately: we could use uv and its lockfile for tracking things and uv export a pip requirements file with the same versions to install from in the container.

@mistydemeo mistydemeo force-pushed the misty/locked_deps branch 3 times, most recently from 6ad72c0 to 40405c1 Compare May 12, 2026 17:38
@mistydemeo
Copy link
Copy Markdown
Contributor Author

Accidentally merged the dev and test groups here; I can split those back out if we want them separate.

@mistydemeo mistydemeo force-pushed the misty/locked_deps branch 2 times, most recently from e14ec88 to 08b6116 Compare May 12, 2026 18:23
@mistydemeo mistydemeo force-pushed the misty/locked_deps branch 2 times, most recently from 1aad2b1 to 48fe822 Compare May 13, 2026 16:36
I chose to go with uv's lockfile format for convenience since that lets
us track the dependency tree a little more easily.
@mistydemeo mistydemeo force-pushed the misty/locked_deps branch from 48fe822 to c88f107 Compare May 13, 2026 21:14
Copy link
Copy Markdown
Member

@emma-sg emma-sg left a comment

Choose a reason for hiding this comment

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

In favour of this! It seems to speed things up a fair bit, and it also would mean we'd be able to use exclude-newer as a layer of supply-chain security.

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