Skip to content

Widen omop-alchemy, oa-configurator, and omop-emb pins to semver ranges - #33

Closed
nicoloesch wants to merge 3 commits into
mainfrom
32-dep-version-range
Closed

Widen omop-alchemy, oa-configurator, and omop-emb pins to semver ranges#33
nicoloesch wants to merge 3 commits into
mainfrom
32-dep-version-range

Conversation

@nicoloesch

@nicoloesch nicoloesch commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Now that #28 put this repo and its upstream packages on label-driven semver, replaces exact pins with >=current,<next-major ranges across all three sibling deps (omop-alchemy, oa-configurator, omop-emb, including the faiss-cpu/pgvector extras). Also aligns the dev extra's oa-configurator[postgres] floor with the main pin, and makes the postgres extra's omop-alchemy[postgres] bound explicit. Verified via uv lock this was already effectively bounded through the main deps entry (PEP 508 resolvers merge multiple requirement lines for the same package), so this is a readability change, not a functional one.

Also installs the cava-devops pre-commit hook so future dependency edits keep uv.lock surgically in sync without relying on memory:

chmod +x .githooks/pre-commit
git config core.hooksPath .githooks

Notes

CI on this PR will fail until cava-devops PR AustralianCancerDataNetwork/cava-devops#2 merges to main

  • ci.yml here already points at @main for the --frozen-based build-test.yml, which doesn't exist there yet. This is expected and will resolve on its own once that PR merges; no action needed here beyond waiting.

@nicoloesch nicoloesch added the dependencies Dependency update. PATCH: x.y.z+1 label Jul 23, 2026
@nicoloesch nicoloesch changed the title Widen omop-alchemy, oa-configurator, and omop-emb pins to semver ranges, add min-version CI and Dependabot Widen omop-alchemy, oa-configurator, and omop-emb pins to semver ranges Jul 24, 2026
@nicoloesch

Copy link
Copy Markdown
Collaborator Author

It was decided to absorb this change by #34 to prevent changing the semver ranges for a second time with the imminent BREAKING releases.

@nicoloesch nicoloesch closed this Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Dependency update. PATCH: x.y.z+1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Relax omop-alchemy / oa-configurator / omop-emb pins to semver ranges

1 participant