Skip to content

chore(deps): update multilspy source to FalkorDB org#629

Merged
gkorland merged 1 commit intostagingfrom
chore/update-multilspy-source
Mar 23, 2026
Merged

chore(deps): update multilspy source to FalkorDB org#629
gkorland merged 1 commit intostagingfrom
chore/update-multilspy-source

Conversation

@gkorland
Copy link
Copy Markdown
Contributor

@gkorland gkorland commented Mar 23, 2026

Summary

Switch the multilspy git dependency from AviAvni/multilspy to FalkorDB/multilspy so the project uses the organization-owned fork.

Changes

  • Updated pyproject.toml: changed multilspy URL from github.com/AviAvni/multilspy to github.com/FalkorDB/multilspy (same python-init-params branch)
  • Regenerated uv.lock to reflect the new source

Testing

  • uv lock resolved successfully with the new URL
  • Same package version (v0.0.11), just a different commit source

Memory / Performance Impact

N/A

Related Issues

N/A

Summary by CodeRabbit

  • Chores
    • Updated the source for a project dependency to a new maintainer repository while maintaining compatibility.

Switch multilspy git dependency from AviAvni/multilspy to
FalkorDB/multilspy to use the organization-owned fork.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 23, 2026

📝 Walkthrough

Walkthrough

The multilspy dependency source in pyproject.toml has been updated to point to a different GitHub repository, changing the git URL from AviAvni's fork to the FalkorDB organization while maintaining the same revision reference.

Changes

Cohort / File(s) Summary
Dependency Configuration
pyproject.toml
Updated git repository URL for multilspy dependency from AviAvni's fork to FalkorDB organization, retaining the python-init-params revision reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop, a skip, a URL swap,
From one repo to another we flit,
FalkorDB now holds the key,
Same branch, new home—a simple bit! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: updating the multilspy dependency source from AviAvni's repository to FalkorDB's organization repository.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/update-multilspy-source

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 and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
pyproject.toml (1)

22-22: Consider pinning commit SHA in pyproject.toml for explicitness (optional).

Line 22 references branch @python-init-params, but uv.lock already pins the exact commit c0807fb28788de112bf0a8df7fe9beb45175723b. While the lockfile ensures reproducibility, pinning the commit SHA directly in pyproject.toml makes the intent explicit and is cleaner for dependency transparency.

Suggested change (optional)
-    "multilspy @ git+https://github.com/FalkorDB/multilspy.git@python-init-params",
+    "multilspy @ git+https://github.com/FalkorDB/multilspy.git@c0807fb28788de112bf0a8df7fe9beb45175723b",
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@pyproject.toml` at line 22, The dependency entry for "multilspy" in
pyproject.toml references the branch name "@python-init-params"; update it to
pin the exact commit SHA used in the lockfile
(c0807fb28788de112bf0a8df7fe9beb45175723b) so the pyproject.toml explicitly
matches the reproducible version, i.e., replace the branch ref with the commit
SHA for multilspy to improve transparency and traceability.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@pyproject.toml`:
- Line 22: The dependency entry for "multilspy" in pyproject.toml references the
branch name "@python-init-params"; update it to pin the exact commit SHA used in
the lockfile (c0807fb28788de112bf0a8df7fe9beb45175723b) so the pyproject.toml
explicitly matches the reproducible version, i.e., replace the branch ref with
the commit SHA for multilspy to improve transparency and traceability.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 71ae60d3-a785-4300-8c45-96efed5390e0

📥 Commits

Reviewing files that changed from the base of the PR and between 00724ce and b36a5f0.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • pyproject.toml

@gkorland gkorland merged commit 751d85f into staging Mar 23, 2026
12 checks passed
@gkorland gkorland deleted the chore/update-multilspy-source branch March 23, 2026 13:17
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.

1 participant