Skip to content

fix(lamindb): bump stale version pins to current 2.10.0 - #915

Merged
stanfish06 merged 2 commits into
masterfrom
skill-health/lamindb
Sep 19, 2026
Merged

stanfish06 merged 2 commits into
masterfrom
skill-health/lamindb

Conversation

@stanfish06

Copy link
Copy Markdown
Owner

Summary

  • lamindb/lamindb-core are pinned to stale 2.7.0/2.6.1, three minor releases behind current 2.10.0
  • The "core is out-of-sync, pin separately" caveat is no longer true — both packages ship in lockstep at 2.10.0 per PyPI
  • Updates the version pins and rewrites the caveat accordingly

Fixes #907

Note on generated navigation

This only changes install-command body text (not name/description/frontmatter that drives vault/), so the generated wrapper note is unaffected. I couldn't run ./skillquarium build locally — bun.sh (needed to fetch a Bun ≥1.4.2 matching this repo's lockfile version) is blocked by this sandbox's network policy — but CI's rebuild-index.yml regenerates and commits the nav layer automatically on push to master, consistent with how other recent skill-health/* fix PRs in this repo have shipped.

Test plan

  • Verified current versions independently via the PyPI JSON API (lamindb and lamindb-core both report 2.10.0)
  • Confirmed no other stale version references remain in the file

🤖 Generated with Claude Code

https://claude.ai/code/session_01GSjSNfsbDHtxu6d6FFVEGi


Generated by Claude Code

lamindb and lamindb-core now ship in lockstep at 2.10.0 (per PyPI),
so the "out-of-sync" caveat no longer applies.

Fixes #907

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GSjSNfsbDHtxu6d6FFVEGi
@coderabbitai

coderabbitai Bot commented Sep 19, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 19ca44fa-3a7d-4757-8790-f0145655fc94

📥 Commits

Reviewing files that changed from the base of the PR and between be36d5c and 3fbc17f.

📒 Files selected for processing (2)
  • .skill-vault/data/local-overrides.json
  • skills/lamindb/SKILL.md

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@stanfish06

Copy link
Copy Markdown
Owner Author

The "Code scanning AI findings" check (dynamic/agents/github-advanced-security) failed on this PR. It's not caused by this diff: the job log shows it dies before analyzing any files — SessionModelError: CAPIError: 400 The requested model is not supported from GitHub's Copilot Autofix backend, a repo-side infra/config issue. The same failure reproduces identically on the pre-existing, unrelated PR #906. The substantive Claude Code Review check passed. Not fixing this here since it's outside this PR's scope — flagging so it isn't mistaken for a content problem.


Generated by Claude Code

lamindb is upstream-managed (K-Dense-AI/scientific-agent-skills) and the
next sync would revert this PR (#894). Upstream still pins 2.5.1, so the
`find` string is upstream's block rather than master's 2.7.0 text; the
entry is already applied in this tree and re-applies after a sync.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@stanfish06
stanfish06 marked this pull request as ready for review September 19, 2026 18:25
@stanfish06
stanfish06 merged commit 2f555a6 into master Sep 19, 2026
5 of 6 checks passed
@stanfish06
stanfish06 deleted the skill-health/lamindb branch September 22, 2026 17:43
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.

[skill-health] lamindb: pinned 2.7.0 baseline is 3 minor releases stale, and the "core out-of-sync" caveat no longer applies

2 participants