Skip to content

Bump dependencies#2931

Merged
jlowin merged 2 commits intomainfrom
deps
Jan 19, 2026
Merged

Bump dependencies#2931
jlowin merged 2 commits intomainfrom
deps

Conversation

@jlowin
Copy link
Copy Markdown
Member

@jlowin jlowin commented Jan 19, 2026

No description provided.

@jlowin jlowin added the ignore in release notes Minor change for release notes. Use sparingly for meta PRs like workflow tests. label Jan 19, 2026
@marvin-context-protocol marvin-context-protocol Bot added the dependencies Updates to project dependencies. Automatically applied to dependabot PRs. label Jan 19, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 19, 2026

Walkthrough

This pull request removes inline type: ignore comments from method signatures across multiple modules and refactors the add_to_docket method declarations. Changes include updating method signatures to use keyword-only parameters for fn_key, task_key, and **kwargs in prompts, tools, and resource template classes. Additionally, task registry assignments in the client are modified, and cursor extraction logic in server functions is simplified. The overall behavioral semantics remain unchanged.

Possibly related PRs

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title 'Bump dependencies' is vague and does not accurately reflect the actual changes, which involve removing type: ignore annotations and updating method signatures across multiple files. Revise the title to describe the actual changes, such as 'Remove type: ignore annotations and update method signatures' or check if this is the correct PR.
Description check ⚠️ Warning The pull request description is entirely empty, missing all required sections from the template including description, issue reference, and checklists. Add a comprehensive description following the template, including what changes were made, which issue is being closed, and completed checklist items.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

@jlowin jlowin merged commit 71d6f49 into main Jan 19, 2026
11 checks passed
@jlowin jlowin deleted the deps branch January 19, 2026 13:38
gfortaine pushed a commit to gfortaine/fastmcp that referenced this pull request Jan 30, 2026
gfortaine pushed a commit to gfortaine/fastmcp that referenced this pull request Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Updates to project dependencies. Automatically applied to dependabot PRs. ignore in release notes Minor change for release notes. Use sparingly for meta PRs like workflow tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant