Skip to content

Move linkml to runtime dependencies to prevent version skew#192

Merged
amc-corey-cox merged 2 commits intomainfrom
fix-linkml-runtime-dep
Apr 7, 2026
Merged

Move linkml to runtime dependencies to prevent version skew#192
amc-corey-cox merged 2 commits intomainfrom
fix-linkml-runtime-dep

Conversation

@amc-corey-cox
Copy link
Copy Markdown
Contributor

Summary

  • Move linkml>=1.10.0 from dev dependencies to runtime dependencies
  • data_loaders.py imports linkml.validator.loaders at runtime, so downstream users need the constraint
  • Without it, uv can resolve linkml-runtime 1.10.0 alongside linkml 1.9.x, causing Format.JSON import errors

Test plan

  • 553 tests pass
  • CI passes

🤖 Generated with Claude Code

linkml is imported at runtime (data_loaders.py uses
linkml.validator.loaders) but was only in dev dependencies.
Downstream users got linkml-runtime>=1.10.0 without a matching
linkml constraint, allowing uv to resolve an older linkml that's
incompatible with runtime 1.10.0 (Format.JSON rename).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 7, 2026 14:31
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR ensures linkml is installed for downstream users by promoting it to a runtime dependency, preventing version skew between linkml and linkml-runtime that can cause runtime import errors (notably around JSON format handling).

Changes:

  • Add linkml>=1.10.0 to runtime dependencies in pyproject.toml.
  • Regenerate uv.lock so the published requires-dist and editable package dependency set includes linkml.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
pyproject.toml Adds linkml>=1.10.0 to runtime dependencies so end users get linkml installed.
uv.lock Updates the locked metadata/dependency lists to reflect linkml as a runtime requirement.

Comment thread pyproject.toml
@amc-corey-cox amc-corey-cox merged commit 1639e8c into main Apr 7, 2026
8 checks passed
@amc-corey-cox amc-corey-cox deleted the fix-linkml-runtime-dep branch April 7, 2026 14:35
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