Skip to content

Conversation

@dsarno
Copy link
Collaborator

@dsarno dsarno commented Oct 21, 2025

fix previous test PR, incorrect pytest dep

Summary by CodeRabbit

  • Bug Fixes

    • Improved branch name display to show actual branch references instead of generic labels.
  • Chores

    • Updated development dependencies for testing framework.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 21, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Dependencies updated in the dev group (pytest-anyio replaced with pytest-asyncio). Source configuration enhanced to display dynamic branch names in option labels and adjusted upstream URL scheme from git+https to plain https.

Changes

Cohort / File(s) Summary
Dev Dependencies
MCPForUnity/UnityMcpServer~/src/pyproject.toml
Replaced pytest-anyio>=0.6.0 with pytest-asyncio>=0.23 in optional dev dependencies.
Dynamic Branch Labels & URL Scheme
mcp_source.py
Option labels (2 and 3) now interpolate actual branch names using f-strings instead of static text. Upstream URL changed from git+https scheme to plain https scheme.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A rabbit's ode to branches bright,
No more static text in sight!
From git+https to https plain,
pytest-asyncio runs the vein,
Dynamic names make config right! 🌿

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0397887 and eac7c6c.

📒 Files selected for processing (2)
  • MCPForUnity/UnityMcpServer~/src/pyproject.toml (1 hunks)
  • mcp_source.py (2 hunks)

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.

@dsarno dsarno merged commit e3e5485 into CoplayDev:main Oct 21, 2025
1 check was pending
@dsarno dsarno deleted the fix/replace-pytest-anyio-with-pytest-asyncio branch October 24, 2025 17:40
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