Skip to content

cleanup: MCP extra include CORE#104

Merged
julpayne merged 1 commit intoeval-hub:mainfrom
tarilabs:tarilabs-20260330-cleanups
Mar 30, 2026
Merged

cleanup: MCP extra include CORE#104
julpayne merged 1 commit intoeval-hub:mainfrom
tarilabs:tarilabs-20260330-cleanups

Conversation

@tarilabs
Copy link
Copy Markdown
Member

@tarilabs tarilabs commented Mar 30, 2026

What and why

avoid duplication and make linear

Type

  • feat
  • fix
  • docs
  • refactor / chore
  • test / ci

Testing

  • Tests added or updated
  • Tested manually

Breaking changes

none

Summary by CodeRabbit

  • Chores
    • Streamlined the CLI optional dependency to remove an unnecessary extra, reducing installation footprint and simplifying the CLI install experience. No other configuration or dependency groups were changed.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 30, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 9169a3c7-effc-4b34-b65f-14001388f17d

📥 Commits

Reviewing files that changed from the base of the PR and between 967d0db and 0424eb2.

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

📝 Walkthrough

Walkthrough

The cli optional dependency group in pyproject.toml was changed to remove the core extra from the eval-hub-sdk dependency, updating eval-hub-sdk[core,mcp] to eval-hub-sdk[mcp]. Only one line was modified.

Changes

Cohort / File(s) Summary
Dependency Configuration
pyproject.toml
Updated cli optional dependency group: removed core extra from eval-hub-sdk specification, now references eval-hub-sdk[mcp].

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 I nudged out "core" with a gentle hop,
Now mcp stays in its tidy little spot.
One line trimmed, the meadow feels light,
I twitch my whiskers and bound out of sight. 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title directly relates to the main change: removing the 'core' extra from the CLI dependency group, keeping only 'mcp' to avoid duplication.
Description check ✅ Passed The description includes all required template sections: 'What and why', 'Type', 'Testing', and 'Breaking changes', though some sections lack detailed information.
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

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.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: tarilabs <matteo.mortari@gmail.com>
@julpayne julpayne merged commit ba08ff0 into eval-hub:main Mar 30, 2026
9 checks passed
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